Re: APR_POOL_DEBUG is functioning again WAS: RE: Don't use APR_POOL_DEBUG

2002-01-15 Thread Jeff Trawick
"Sander Striker" <[EMAIL PROTECTED]> writes: > Hi, > > As of a recent commit the apr pools debug code is > functioning again. I tested it with httpd-2.0 HEAD > and it works (both tried worker and prefork to see > if everything goes to the log ok). I've tried worker with the latest apr_pools.c.

Re: [PATCH] Add --enable-pool-debug to configure

2002-01-15 Thread Jim Jagielski
Sander Striker wrote: > > Ok, that sounds like a plan. > One minor nit which would be great if we could agree on: > simply defining APR_POOL_DEBUG should enable the basic debug > mode. > +1... Most likely it would be best to do the mojo in apr_pools.h and adjust apr_pools.c accordingly. For exa

RE: [PATCH] Export and document apr_pool_xxx_debug

2002-01-15 Thread Sander Striker
> From: Sander Striker [mailto:[EMAIL PROTECTED] > Subject: [PATCH] Export and document apr_pool_xxx_debug > > Hi, > > Projects that are always calling apr_pool_create, > clear or destroy from a wrapper and have a debug > version of that wrapper could benefit from this > patch. One project direc

[PATCH] Export and document apr_pool_xxx_debug

2002-01-15 Thread Sander Striker
Hi, Projects that are always calling apr_pool_create, clear or destroy from a wrapper and have a debug version of that wrapper could benefit from this patch. One project directly comes to mind: subversion. Exporting the apr_pool_xxx_debug functions (which technically is already happening) will r

APR_POOL_DEBUG is functioning again WAS: RE: Don't use APR_POOL_DEBUG

2002-01-15 Thread Sander Striker
Hi, As of a recent commit the apr pools debug code is functioning again. I tested it with httpd-2.0 HEAD and it works (both tried worker and prefork to see if everything goes to the log ok). Some hints (that probably change when we move to a numbered system for selecting the debug mode): If you

RE: [PATCH] Add --enable-pool-debug to configure

2002-01-15 Thread Sander Striker
> From: Justin Erenkrantz [mailto:[EMAIL PROTECTED] > Subject: Re: [PATCH] Add --enable-pool-debug to configure > > On Mon, Jan 14, 2002 at 11:55:48AM -0800, Roy T. Fielding wrote: > > > +AC_ARG_ENABLE(pool-debug, > > > + [ --enable-pool-debug[={yes|verbose}]], > > > + [ if test -z "$enableval"

Re: [PATCH] Add --enable-pool-debug to configure

2002-01-15 Thread Justin Erenkrantz
On Mon, Jan 14, 2002 at 11:55:48AM -0800, Roy T. Fielding wrote: > > +AC_ARG_ENABLE(pool-debug, > > + [ --enable-pool-debug[={yes|verbose}]], > > + [ if test -z "$enableval" -o "$enableval" = "yes"; then > > +APR_ADDTO(CPPFLAGS, -DAPR_POOL_DEBUG) > > +elif test "$enableval" = "verbos