apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Cliff Woolley <cliffwool...@yahoo.com>
Subject Re: [PATCH] APR_DEBUG_BUCKET option
Date Fri, 14 Dec 2001 07:17:07 GMT

   Hmmm... it's too bad that this adds so many #ifdefs to code that's
largely #ifdef free...  I guess I can see some instances where this would
be helpful, but I'm not sure it's worth cluttering up all the buckets code
for.  I'm trying to think of another way to get roughly the same
information without all the mess.  What if we did it at bucket-insertion
time?  That way you'd only have to change the brigade insert macros.  A
bucket that never got inserted into a brigade can't hurt you (except to
leak memory ;) anyway.  And actually, knowing when a particular bucket got
stuck in the brigade it's currently in is also valuable information...


   Cliff Woolley
   Charlottesville, VA

View raw message