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

--Cliff


--------------------------------------------------------------
   Cliff Woolley
   cliffwoolley@yahoo.com
   Charlottesville, VA



Mime
View raw message