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

Ian MacLarty maclarty at cs.mu.OZ.AU
Fri May 20 12:47:18 AEST 2005


On Fri, 20 May 2005, Julien Fischer wrote:

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

Okay.

Ian.

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