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

[Paul Bone]

On Mon, Oct 07, 2013 at 11:53:39AM +1100, Julien Fischer wrote:
>
> 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.

It makes more sense to me, they're seperate changes that happen to be
related.  Especially that this patch preceeds the fix.

>> 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?

I assume so.  I include both because I wrote the other one first and didn't
see any point removing it after I added this one.

>>
>> 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.

Okay, that's relevant to the other patch.

Thanks.


-- 
Paul Bone
http://www.bone.id.au