[m-rev.] for review: Do not escape character strings in internal atom representation.
Julien Fischer
jfischer at opturion.com
Mon May 26 15:10:05 AEST 2014
On Mon, 26 May 2014, Peter Wang wrote:
> Branches: master, 14.01
>
> The internal representation of a character constant as an atom term
> should not use escaped character syntax; the escaping should occur
> when writing the external representation only.
>
> compiler/hlds_out_mode.m:
> compiler/intermod.m:
> compiler/prog_util.m:
> compiler/type_constraints.m:
> compiler/typecheck.m:
> Fix instances of the bug.
>
> tests/valid/Mercury.options
> tests/valid/Mmakefile
> tests/valid/intermod_char.m
> tests/valid/intermod_char2.m
> Add test case. The .opt file was invalid because the character
> in the clause head was doubly-escaped.
That looks fine.
Cheers,
Julien.
More information about the reviews
mailing list