tests: expand testing of augmented assignment operators - Githubissues

crashappsec / libcon4m

Base Compiler and Runtime Support for con4m

Apache License 2.0

0 stars 0 forks source link

tests: expand testing of augmented assignment operators #70

Closed ee7 closed 1 week ago

ee7 commented 2 weeks ago

Add some more tests for augmented assignment operators, and remove some existing ones that had similar coverage.

assignops.c4m was similar to basic10.c4m:

https://github.com/crashappsec/libcon4m/blob/b4fdc8e2e70b0ef18a554562702e2db1e0c9aff0/tests/basic10.c4m#L1-L15

The grammar specifies the augmented assignment operators here:

https://github.com/crashappsec/libcon4m/blob/b4fdc8e2e70b0ef18a554562702e2db1e0c9aff0/doc/reference.md#L443-L445

Note that commit https://github.com/crashappsec/libcon4m/commit/b4fdc8e2e70b0ef18a554562702e2db1e0c9aff0 changed the behavior of the /= operator so that:

For division by zero, SIGFPE is no longer produced.
For division by a negative number, the result is now correct.

Refs: https://github.com/crashappsec/libcon4m/issues/55 Refs: https://github.com/crashappsec/libcon4m/issues/57 Refs: https://github.com/crashappsec/libcon4m/issues/71