[m-dev.] For review: another fix for remote CVS

Fergus Henderson fjh at cs.mu.OZ.AU
Tue May 25 13:26:35 AEST 1999


On 25-May-1999, Warwick Harvey <wharvey at cs.monash.edu.au> wrote:
> CVSROOT/check.pl:
> 	Make sure we print an error message unless the return code from
> 	`xmessage' is explicitly either `0' or `1'.  Also, swap the return
> 	values from `xmessage' so that `yes' returns `0' rather than `1',
> 	to further reduce the chance that an error (non-zero exit status)
> 	is interpreted as confirmation to proceed.

That looks fine -- and as far as readability goes it's a definite
improvement on the original code.

Cheers,
	Fergus.

-- 
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.
--------------------------------------------------------------------------
mercury-developers mailing list
Post messages to:       mercury-developers at cs.mu.oz.au
Administrative Queries: owner-mercury-developers at cs.mu.oz.au
Subscriptions:          mercury-developers-request at cs.mu.oz.au
--------------------------------------------------------------------------



More information about the developers mailing list