Andrei Pelinescu-Onciul wrote: > andrei 2008/04/15 18:14:56 CEST > > SER CVS Repository > > Modified files: (Branch: rel_2_0_0) > . globals.h socket_info.h sr_module.h > tls_hooks.h > Log: > - fixes: missing extern for variables declared in *.h files (can cause > different symbols with the same name => unexpected bugs) >
I suggest adding -Wshadow to gcc options, which can warn about nasty bugs then you will get warnings like: In file included from action.c:64: ut.h: In function ‘str2s’: ut.h:155: warning: declaration of ‘str’ shadows a global declaration str.h:38: warning: shadowed declaration is here ut.h: In function ‘string2hex’: ut.h:325: warning: declaration of ‘str’ shadows a global declaration str.h:38: warning: shadowed declaration is here ut.h: In function ‘pathmax’: /alfred > Revision Changes Path > 1.67.2.4 +14 -11 sip_router/globals.h > http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/sip_router/globals.h.diff?r1=1.67.2.3&r2=1.67.2.4 > 1.7.2.1 +4 -4 sip_router/socket_info.h > http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/sip_router/socket_info.h.diff?r1=1.7&r2=1.7.2.1 > 1.40.2.1 +2 -2 sip_router/sr_module.h > http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/sip_router/sr_module.h.diff?r1=1.40&r2=1.40.2.1 > 1.1.2.1 +2 -2 sip_router/tls_hooks.h > http://cvs.berlios.de/cgi-bin/viewcvs.cgi/ser/sip_router/tls_hooks.h.diff?r1=1.1&r2=1.1.2.1 > _______________________________________________ > Serdev mailing list > [email protected] > http://lists.iptel.org/mailman/listinfo/serdev _______________________________________________ Serdev mailing list [email protected] http://lists.iptel.org/mailman/listinfo/serdev
