apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Cliff Woolley <cliffwool...@yahoo.com>
Subject Re: cvs commit: apr/memory/unix apr_sms.c
Date Sat, 19 May 2001 21:48:14 GMT
On Sat, 19 May 2001, Ben Laurie wrote:

> > Just to verify (haven't looked at this section of the code itself yet),
> > assignment IS what's intended here, right?  If so, a ((foo = bar) != NULL)
> > might make that more clear.
> Good point. I foolishly assumed the assignment was intentional - it
> still looks like it is, but confirmation from someone who knows would be
> good. I agree with != NULL if it is so.

Looking closer, I agree.  Confirmation would still be nice, but I
agree that it looks intentional.  So I'll go ahead and throw a !=NULL in


   Cliff Woolley
   Charlottesville, VA

View raw message