CVS commit: src/tests/bin/sh

["Robert Elz" <kre%netbsd.org@localhost>]

Module Name:    src
Committed By:   kre
Date:           Sat Dec 13 08:11:18 UTC 2025

Modified Files:
        src/tests/bin/sh: t_arith.sh

Log Message:
Avoid using optional features in an unrelated test

In the additional tests just added, a use of the post-increment
++ operator was added.  That was not the right thing to do,
as that operator isn't required to be implemented, and if not
would appear as if a ',' operator test had failed.

The actual tests of the ++ and -- operators verify that the shell
being tested actually implements them, before running the tests.
Doing that here would be overly burdensome, so instead just
use i += 1 (which is required) instead of i++ which is not.  For
the purposes of the test it makes no difference (i ends up 1 bigger).

Note that this was not the (sub-)test that will currently fail, the
NetBSD shell does support ++ (and --) both pre & post, which is
why I didn't initially think of this issue.


To generate a diff of this commit:
cvs rdiff -u -r1.9 -r1.10 src/tests/bin/sh/t_arith.sh

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.