[m-rev.] for review: make some io procedures return "" instead of NULL

Julien Fischer juliensf at cs.mu.OZ.AU
Fri May 20 12:25:36 AEST 2005


On Fri, 20 May 2005, Ian MacLarty wrote:

> On Fri, 20 May 2005, Julien Fischer wrote:
>
> >
> > On Thu, 19 May 2005, Ian MacLarty wrote:
> > > Index: library/io.m
> > > ===================================================================
> >
> > That's fine except that I think you should put a brief comment
> > at each of those three spots explaining why "" is returned rather
> > than NULL.
> >
> How about:
> 		/*
> 		** Return a real string, otherwise mdb will break when it tries
> 		** to print the string.
> 		*/
or:
	We can't just return NULL here, because otherwise mdb will break
	when it tries to print the string.

Julien.

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



More information about the reviews mailing list