Sander Striker wrote:
From: Ian Holsman [mailto:[EMAIL PROTECTED]
Subject: Re: [PATCH] Default tagging of pools in debug mode [2]

Stupid Question Time:
is there a problem just enabling file/line passing to this ALL the time?
I mean.. it's just a extra parameter on the stack.

this would make it MUCH easier to enable/disable debugging.


Although it would make things easier to enable debugging, for normal operation of an apr using application it is not needed.

true.
but it doesn't affect performance or functionality or usability.
what this would enable us to do would be to change the .so/dll and
bang.. instant debug info without haveing to recompile anything.

this would be MUCH easier to manage when you are experiencing a problem
with a live/production site.

I don't feel the current hurdle one has to take to do memory
debugging is too high.

I'm -1 on doing file/line passing by default.  It is a debugging
tool/feature and it should stay that.

I'm not arguing that it is a debugging tool. I am saying that by passing
the args and NOT doing anything with them you are making it easier to enable debugging.
your way I need to have a debug version of APR/APR-UTIL/HTTPD sitting around to debug. the other method i just need a debug version of APR.
(as the include file DOESNT CHANGE)



Sander






Reply via email to