[m-dev.] for review: new debugger command set (part 1 of 4)
    Fergus Henderson 
    fjh at cs.mu.OZ.AU
       
    Fri Oct  2 19:57:37 AEST 1998
    
    
  
> *** UPDATE TEST CASES ***
> 
> tests/debugger/Mmakefile:
> 	Reenable queens. Conform to the new set of options.
> tests/debugger/*.inp:
> tests/debugger/*.exp:
> 	Update the inputs and expected outputs of the debugger test cases
> 	to use the new command set and output formats.
...
tests/debugger/Mmakefile:
> @@ -1,8 +1,8 @@
>  
> -include ../Mmake.common
> +include		../Mmake.common
I think it would be better to use a space there
rather than a tab, for consistency with all the
other `include' statements in */Mmakefile.
> +# EXTRA_CFLAGS = -g -v
> +# EXTRA_MLFLAGS = -g -v
Those two lines should either be deleted
or documented.
I didn't see any diffs for the tests/debugger/*.{inp,exp} files.
Probably the diffs for the `.exp' files are not worth reading,
but I would like to see the diffs for the `.inp' files.
Apart from that, that part ("UPDATE TEST CASES") looks fine.
-- 
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