[PATCH] Unused variables x and found_macro_x on process_macros_r()

[Ricardo Maraschini]

----- Original Message -----
> 
> 
> ----- Original Message -----
> > On 06/22/2012 08:57 PM, Ricardo Maraschini wrote:
> > > hello,
> > > 
> > > i've removed two unused variables on found_macro_x() function.
> > > Compiler errors:
> > > 
> > > ../common/macros.c: In function ‘process_macros_r’:
> > > ../common/macros.c:133:6: warning: variable ‘found_macro_x’ set
> > > but
> > > not used [-Wunused-but-set-variable]
> > > ../common/macros.c:128:6: warning: variable ‘x’ set but not used
> > > [-Wunused-but-set-variable]
> > > 
> > > 
> > > OK? Comments?
> > > 
> > 
> > Test this with macro resolution debugging turned on. I just
> > reverted a patch that did this change (and a bunch of other
> > -Wunused-but-set-variable), and removing the unused ones
> > actually broke macro resolution for the core.
> 
> Just for curiosity, the patch you've reverted had the same logic as
> mine regarding strcmp()?
> 

Oh, now I see. I've made a diff between revisions 1853 and 1951 for common/macros.c and it clearly pointed me to a bug. That patch(applied to create revision 1951) is different of mine, regarding strcmp() use. Really, that patch is supposed to explode everything, you gonna see it if you look closely :-)

BTW, I've tested the patch i've sent to you and i had no problems regarding macro resolutions so far.

[1340638282.241842] [2048.1] [pid=20560] **** BEGIN MACRO PROCESSING ***********
[1340638282.241848] [2048.1] [pid=20560] Processing: '$USER1$/check_ssh -t 60 -p 54978 $HOSTADDRESS$'
[1340638282.241910] [2048.1] [pid=20560]   Done.  Final output: '/usr/local/opmon/libexec/check_ssh -t 60 -p 54978 192.168.8.1'
[1340638282.251285] [2048.1] [pid=20560] **** BEGIN MACRO PROCESSING ***********

[1340638282.269438] [2048.1] [pid=20560] Processing: '$USER1$/check_nrpe -H $HOSTADDRESS$ -p 5666 -t 60 -u -c $ARG1$ -a $ARG2$'
[1340638282.269463] [2048.1] [pid=20560]   Done.  Final output: '/usr/local/opmon/libexec/check_nrpe -H 127.0.0.1 -p 5666 -t 60 -u -c otrs_tickets_entrada_user -a 0 0 3'
[1340638282.277821] [2048.1] [pid=20560] **** BEGIN MACRO PROCESSING ***********


-rm

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Nagios-devel mailing list
Nagios-devel at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nagios-devel