httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dean Gaudet <dgau...@arctic.org>
Subject Re: ap_strftime
Date Thu, 30 Dec 1999 18:34:27 GMT
i worry only about namespace collision...

Dean

On Thu, 30 Dec 1999, Ryan Bloom wrote:

> On Wed, 29 Dec 1999, Dean Gaudet wrote:
> 
> > > >     ap_status_t ap_strftime(char *s, ap_size_t max, const char
> > > >                         *format, struct atime_t *tm)
> > 
> > struct atime_t ?  shouldn't that be ap_time_t ?
> 
> Nope.  Because we are using incomplete types, I have two basic naming
> conventions.  Inside APR in the library that defines the type, we are
> using the format "struct basetype_t".  Outside of APR, this is defined to
> be "ap_basetype_t", and we use that outside of the APR library.  Yes, I
> could have named the struct ap_basetype_t, but I was hoping that by not
> naming them the same things, people would be forced to look around and
> figure out the incomplete types stuff.  Not sure if this was a good design
> or not, but it has let me find a bug or two very quickly.
> 
> Ryan
> 
> 
> _______________________________________________________________________
> Ryan Bloom		rbb@raleigh.ibm.com
> 4205 S Miami Blvd	
> RTP, NC 27709		
> 
> Come to the first official Apache Software Foundation
> Conference!  <http://ApacheCon.Com/>
> 
> 
> 


Mime
View raw message