[m-dev.] bytecode stuff in cvs repository

Tyson Richard DOWD trd at students.cs.mu.oz.au
Sun Mar 30 13:10:07 AEST 1997


> 
> |Hi Bert,
> |
> |I notice that you checked in a bunch of code into a directory
> |`mercury/bytecode' in the CVS repository.

		...

> 
> Andrew reviewed the code.
> 

You might not have known that there is a review procedure in
mercury/compiler/notes/REVIEWS. 

The review procedure exists to do more than just make sure someone
reviews a change. If there are problems with this procedure, I'd rather
discuss them than just ignore it.

Perhaps the review procedure should be added into the developers web
page.

(This is not the only case recently - there have been a couple of 
quick-fix changes lately that haven't included _what_ was changed,
just that it was a type/mistake/etc. If you aren't going to have them
reviewed, at least add enough information so that people don't have to
go searching to see what you've changed).




More information about the developers mailing list