[m-rev.] for review: warn about the deprecated module qualifier

Julien Fischer juliensf at cs.mu.OZ.AU
Mon Feb 14 13:19:14 AEDT 2005


On Wed, 9 Feb 2005, Julien Fischer wrote:

>
> Estimated hours taken: 2
> Branches: release
>
> Add a warning for the use of the deprecated module qualifier `:'.
> Doing this via a global variable is a bit hacky but it's only
> for the 0.12 release and it's easier than reworking the parser
> to do it properly.
>
> compiler/prog_io.m:
> 	Add a global variable and some impure predicates to get/set
> 	it's value that we can use to record whether the parser
> 	encountered `:' used as a module qualifier.  We can't use the IO
> 	state here because these need to be called from within semidet code.
>
> 	Emit a warning if a module uses `:' as a module qualifier.
>
> 	In error messages about qualified terms that cannot be parsed
> 	don't refer to module qualifier as `:' if it was `.'.
>
> 	Delete a duplicate import of parse_tree.modules.
>
I intend to commit this today if there are no review comments.

Cheers,
Julien.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list