[m-rev.] for test and review by Peter: fix mantis 391

Zoltan Somogyi zoltan.somogyi at runbox.com
Fri Aug 14 22:04:12 AEST 2015


On Fri, 14 Aug 2015 20:28:57 +1000 (AEST), "Zoltan Somogyi" <zoltan.somogyi at runbox.com> wrote:
> Peter, I think this fixes mantis 391, but I can only look at
> the generated .opt file; in the absence of the rest of the code
> that require_lt.m was derived from, I cannot fully test the fix.
> Can you please try out this diff? You may also want to add
> a test case for the bug to our test suite.

Actually, that diff had a slight problem with white space in the output,
which this updated diff fixes. This has passed bootcheck, so I am committing
this, but I am still interested in a review, and a report of whether it
fixes the original problem.

Zoltan.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.bug391a
Type: application/octet-stream
Size: 12154 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20150814/c59c5f32/attachment.obj>


More information about the reviews mailing list