[m-dev.] for review: new debugger command set (part 1 of 4)

Fergus Henderson fjh at cs.mu.OZ.AU
Fri Oct 2 17:16:45 AEST 1998


On 01-Oct-1998, Zoltan Somogyi <zs at cs.mu.OZ.AU> wrote:
> 
> doc/mdbrc.in:
> 	A debugger command script that reads in the online documentation
> 	and then defines some standard aliases.

Hmm, shouldn't that go in the `scripts' directory rather than the `doc'
directory?

> --- code_info.m	1998/08/25 05:09:41	1.228
> +++ code_info.m	1998/09/16 05:07:23
> @@ -64,9 +64,9 @@
>  		% Create a new code_info structure.
>  :- pred code_info__init(varset, set(var), stack_slots, bool, globals,
>  	pred_id, proc_id, proc_info, instmap, follow_vars, module_info,
> -	int, resume_point_info, code_info).
> +	int, resume_point_info, maybe(int), code_info).

You should document the purpose of the new `maybe(int)' parameter.
(Quite a few of the other parameters, in particular the `bool',
`int', and `set(var)' parameters, are also underdocumented.)

-- 
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.



More information about the developers mailing list