On Sun, 28 Oct 2012, Ingo Molnar wrote:
> I left it a bit mystic because in some cases this macro was
> mis-used not to suppress GCC being wrong, but to hide GCC being
> *right*: for example unused variable warnings in cases like:
>
> int uninitialized_var(var);
>
> #ifdef XYZ
> var = ...;
> ...
> #endif
>
> which (ab-)use was no doubt actively dangerous beyond being
> ugly. One such example is in arch/x86/mm/numa.c. (These cases
> now turn into clear (and always harmless) compiler warnings, as
> they should.)
>
I like initializing them to 0 or NULL because it will still emit the
"unused variable" warnings whereas using uninitialized_var() would not
with -Wall. It's quite possible that uninitialized_var() is actually
suppressing this warning for variables that aren't used.
I fixed a bug that was attributed to uninitialized var for rc1 in
43385846968b ("fs, xattr: fix bug when removing a name not in xattr
list"), so thanks very much for removing it entirely.
------------------------------------------------------------------------------
Monitor your physical, virtual and cloud infrastructure from a single
web console. Get in-depth insight into apps, servers, databases, vmware,
SAP, cloud infrastructure, etc. Download 30-day Free Trial.
Pricing starts from $795 for 25 servers or applications!
http://p.sf.net/sfu/zoho_dev2dev_nov
_______________________________________________
Openipmi-developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openipmi-developer
