[m-rev.] Documentation and test case for abstract typeclasses.

[Fergus Henderson]

On 31-Oct-2002, Ralph Becket <rafe at cs.mu.OZ.AU> wrote:
> Fergus Henderson, Thursday, 31 October 2002:
> > 
> > Sorry, I did a bad job of reviewing this one.
> > 
> > First, those two changes both require each other, and both constitute part
> > of a single logical change -- they should have been listed as a single
> > change, not two separate changes.  It's OK to put the documentation
> > change separately, but please do not include more than one change in a
> > single email.
> 
> Righto.
> 
> > Second, the test case should go in tests/hard_coded/typeclasses,
> > not in a new directory tests/typeclasses.
> 
> The directory tests/typeclasses was already there, but unpopulated.

No, it wasn't.  Empty directories don't exist, because CVS doesn't
support them.  If you think they do, it's because you are not using
the right CVS options.  You need to use the "-P" option when doing
a cvs checkout or cvs update.
Put 

	update -d -P
	checkout -P

in your ~/.cvsrc.

If you don't use -P, then you will get all kinds of bogus empty directories
which are just historical remnents of past mistakes.

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
The University of Melbourne         |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------