[m-rev.] for review 1/2: Add test cases for bug300

Julien Fischer jfischer at opturion.com
Mon Oct 7 11:53:39 AEDT 2013


Hi,

On Wed, 2 Oct 2013, Paul Bone wrote:

> Add test cases for bug300

I find it inconvenient that this diff is separate from the one that
actually fixes the bug.

> tests/valid/bug300.m:
> tests/valid/Mmakefile:
>    Add simple test case
>
> tests/valid/Mercury.options:
>    Run the new test case with --optimize-constructor-last-call
>
> tests/hard_coded/bug300.m:
> tests/hard_coded/bug300.exp:
> tests/hard_coded/Mmakefile:
>    This test case can detect the bug in C grades, where as the test case in
>    tests/valid/bug300.m only works in Java grades.

Presumably, this one also triggers the bug in Java grades?  If so, why
do we need two test cases?
>
> tests/hard_coded/Mercury.options:
>    Setup MCFLAGS so that the test case can detect the bug.

You should also mention that the bug has been fixed in the NEWS file for
13.05.2.

Cheers,
Julien.



More information about the reviews mailing list