[m-dev.] for review: IL backend bugfixes

Fergus Henderson fjh at cs.mu.OZ.AU
Sun Oct 22 16:40:45 AEDT 2000


On 22-Oct-2000, Tyson Dowd <trd at cs.mu.OZ.AU> wrote:
> diff -u -r1.13 ml_elim_nested.m
> @@ -1384,7 +1393,10 @@
>  			local_vars :: list(mlds__defn),
>  				
>  				% Type of the introduced environment struct
> -			env_type_name :: mlds__type
> +			env_type_name :: mlds__type,
> +
> +				% Type of the introduced environment struct
> +			env_ptr_type_name :: mlds__type

The comment for the second one is a duplicate of the
comment for the first one, which is confusing.

> +
> +:- func this_file = string.
> +this_file = "mlds_to_ilasm.m".

I suggest you add `:- end_module mlds_to_ilasm.m' here.
That way, if someone changes the name of this module, and doesn't
change this code, the compiler will warn them.

[to be continued]

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
WWW: <http://www.cs.mu.oz.au/~fjh>  |  of excellence is a lethal habit"
                                    |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-developers mailing list
Post messages to:       mercury-developers at cs.mu.oz.au
Administrative Queries: owner-mercury-developers at cs.mu.oz.au
Subscriptions:          mercury-developers-request at cs.mu.oz.au
--------------------------------------------------------------------------



More information about the developers mailing list