Le 12/07/2012 13:35, Jeff Squyres a écrit :
> I silenced some harmless compiler warnings (i.e., the compiler couldn't tell 
> that some variables would always be initialized) in r4610.
>
> I'm getting a few others, though -- could you have a look?
>
> distances.c: In function 'hwloc_distances_set_from_env':
> distances.c:259: warning: '__old_locale' may be used uninitialized in this 
> function
> topology-xml.c: In function 'hwloc_topology_export_xmlbuffer':
> topology-xml.c:882: warning: '__old_locale' may be used uninitialized in this 
> function
> topology-xml.c: In function 'hwloc_topology_export_xml':
> topology-xml.c:857: warning: '__old_locale' may be used uninitialized in this 
> function
>

I saw some similar new compiler warnings with gcc 4.7.x (and I already
fixed them) but I can't get yours with 4.7.1 now. Which compiler do you
have?

The patch should fix them but the compiler shouldn't be that dumb.

Brice

--- include/private/private.h   (révision 4610)
+++ include/private/private.h   (copie de travail)
@@ -346,7 +346,7 @@
 #ifdef HAVE_XLOCALE_H
 #include "xlocale.h"
 #endif
-#define hwloc_localeswitch_declare locale_t __old_locale, __new_locale
+#define hwloc_localeswitch_declare locale_t __old_locale = (locale_t)0, 
__new_locale
 #define hwloc_localeswitch_init() do {                     \
   __new_locale = newlocale(LC_ALL_MASK, "C", (locale_t)0); \
   if (__new_locale != (locale_t)0)                         \


Reply via email to