[m-dev.] for review: test suite changes.

Fergus Henderson fjh at cs.mu.OZ.AU
Sun Aug 30 12:22:18 AEST 1998


On 29-Aug-1998, Tyson Dowd <trd at cs.mu.OZ.AU> wrote:
> debugger/Mmakefile:
> 	Disable the queens test, which is has output that doesn't
> 	seem to match its input.
> 
> invalid/Mmakefile:
> 	Generate dependencies for nested_impl_in_int, but don't
> 	try to build it any further. 
> 	This detects the error, but avoids trying to build the entire
> 	program.

Those changes look OK, but I do have a couple of minor suggestions.
One is that some comments in invalid/Mmakefile would be helpful.
The other is a typo in a comment:

> Index: debugger/Mmakefile
> +# Commented out tests are currently disabled because they fail, but are
> +# not vital at the moment because the debugger is still a work in
> +# progres.

s/progres/progress/

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