[PATCH 1/1] trends-add-forward-to-next-problem.patch

Michael Friedrich michael.friedrich at univie.ac.at
Fri Jan 7 12:47:06 CET 2011


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: 	michael.friedrich at univie.ac.at
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


------------------------------------------------------------------------------
Gaining the trust of online customers is vital for the success of any company
that requires sensitive data to be transmitted over the Web.   Learn how to 
best implement a security strategy that keeps consumers' information secure 
and instills the confidence they need to proceed with transactions.
http://p.sf.net/sfu/oracle-sfdevnl 




More information about the Developers mailing list