httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ben Reser <>
Subject Re: Do pools lead to bad programming?
Date Thu, 12 Dec 2013 00:36:18 GMT
On 12/11/13 4:00 PM, Kean Johnston wrote:
> Am I being too obsessive? If not, would you like patches to correct these as I
> find them, and if so, should I open a bug about this or just post patches here
> (they are all likely to be a simple move of 1 or 2 lines)?

There are two ways this sort of thing can happen.

The person who wrote the original code didn't feel like breaking the
declaration of the variable and the allocation into two lines.  Which is a mistake.

Alternatively, the allocation could have always been necessary and the only
difference was the ordering when it happened.  I.E. there may have not been any
returns before the use of the variable.  This would be an error on the part of
the person changing the code.

Looking at this specific case, the variable was added in r357444 as part of the
original template of mod_proxy_fcgi.  So it seems like an error on the original
developers part.

It certainly seems worthwhile to clean these up when they are found in my opinion.

I wouldn't say that pools lead to bad programming.  It's mostly that pools
limit the consequences of these sorts of mistakes.  The mistakes are going to
happen regardless.

View raw message