[m-rev.] for review: diagnose non-contiguous clauses

[Zoltan Somogyi]

On 18-Aug-2009, Julien Fischer <juliensf at csse.unimelb.edu.au> wrote:
>> For review by anyone. Anyone have better names for the options that aren't
>> excessively long?
>
> The names do not appear to be excessively long, at least in comparison
> to some of those that already exist.

I know they are not too long, but that is because I changed the more
descriptive

--warn-non-contiguous-clauses-and-foreign-procs

into just

--warn-non-contiguous-foreign-procs

which is slightly misleading.

>> This option is enabled by default.
>
> That's fine, but until the installed compilers understand the new
> options can you please disable it by default.

Done.

> I am not saying that this should be fixed as part of this change, but
> it is worth bearing in mind that the "let's throw some of the
> definitions away early" approach will eventually need to change.

I have add an XXX comment to say this at the relevant site.

Zoltan.
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------