Page 1 of 1

Re: [Nagios-devel] [PATCH

Posted: Fri Jan 07, 2011 11:46 am
by Guest
Andreas Ericsson wrote:
> On 01/07/2011 09:24 AM, Michael Friedrich wrote:
>> Andreas Ericsson wrote:
>>> On 01/03/2011 03:13 PM, Carsten Emde wrote:
>>>
>>> Applied and pushed. Thanks
>>>
>> Nice patch, thanks. It contains several inline declarations which i have
>> reworked while applying against Icinga CGIs. Attached is a git patch against
>> current Nagios HEAD (from git.op5.org, thx for the mirror) resolving those
>> things too.
> This patch is kinda backwards. It's considered best practice to have the
> variables declared inside the outmost block they're used, so as to prevent
> value spilling when the code is next modified.

It is?

Ok then I'm kinda old school in this regard, so NAK ACK ;-)

> So thanks, but NAK.
>


--
DI (FH) Michael Friedrich

Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria

email: [email protected]
phone: +43 1 4277 14359
fax: +43 1 4277 14338
web: http://www.univie.ac.at/zid
http://www.aco.net

Icinga Core& IDOUtils Developer
http://www.icinga.org






This post was automatically imported from historical nagios-devel mailing list archives
Original poster: [email protected]