httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Marc Slemko <ma...@worldgate.com>
Subject Re: New compile warning this AM
Date Sat, 21 Mar 1998 02:05:49 GMT
On Fri, 20 Mar 1998, Rodent of Unusual Size wrote:

> Marc Slemko wrote:
> > 
> > Uhuh.  And Dean says we should be telling printf the right type to use
> > instead of bogusly casting what we do have.  If we had a configuration
> > system that can know what type getpid() returns, then it is easy
> > to setup the right printf format strings for it.  This is useful
> > in general, not just for this.
> 
> Yes, excellent.  But not for 1.3.  Can we put the cast in for 1.3, and
> make a note to fix this for 2.0?  Either in apache-2.0/STATUS or as a
> suspended PR, I don't care - but it should compile.
> 
> We've always treated getpid() as returning an unsigned long, haven't
> we?  In which case the cast is changing nothing for the worse - so

No, we always treat it as whatever the heck we feel like at the moment,
often int which could be more bogus.

> let's put it in so the bloody software compiles warning-free until
> we can do it correctly.  We can get rid of the compilation warnings
> even if we can't stomp all the bugs; "ye have bugs with you always."

I shall die before I cast a pid to unsigned long on a platform where it
may not be valid!




> 
> #ken	P-)}
> 
> Ken Coar                    <http://Web.Golux.Com/coar/>
> Apache Group member         <http://www.apache.org/>
> "Apache Server for Dummies" <http://WWW.Dummies.Com/
> 


Mime
View raw message