[m-dev.] For review: implementation of collect for Opium-M

Fergus Henderson fjh at cs.mu.OZ.AU
Wed Nov 3 03:07:11 AEDT 1999


On 02-Nov-1999, Erwan Jahier <Erwan.Jahier at irisa.fr> wrote:
> Index: trace/mercury_trace_external.c
..
> +static void	MR_get_object_file_name(Word debugger_request, 
> +			String *objet_file_name_ptr);

s/objet/object/

> +		case MR_collecting:
> +			/* 
> +			** XXX Add a another request that takes 
> +			** arguments into account. We need two kinds 
> +			** of request in order to not penalize the 
> +			** performance of collect in the cases where 
> +			** arguments are not used.
> +			**  
> +			** arguments = MR_make_var_list(layout, saved_regs);
> +			*/
> +			MR_COLLECT_filter(
> +				*filter_ptr, 
> +				seqno, 
> +				depth, 
> +				port,
> +				layout, 
> +				path);

You might as well write that function call on two lines rather than 7.

Apart from that, it looks fine this time, so if it works, please commit it.

-- 
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"
PGP: finger fjh at 128.250.37.3        |     -- 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