gallium: In option helpers, move assignment outside of if clause.
authorVinson Lee <vlee@vmware.com>
Sat, 24 Apr 2010 05:06:19 +0000 (22:06 -0700)
committerVinson Lee <vlee@vmware.com>
Sat, 24 Apr 2010 05:06:19 +0000 (22:06 -0700)
This silences Coverity assign_where_compare_meant warnings.

src/gallium/auxiliary/util/u_debug.h

index ec3371a9b2565c181a9d564f2dbd33a6cfa39d32..e8ff2773e697abcc0f4a59d2ace6c9933a2e9741 100644 (file)
@@ -309,8 +309,10 @@ debug_get_option_ ## sufix (void) \
 { \
    static boolean first = TRUE; \
    static boolean value; \
-   if (first && !(first = FALSE)) \
+   if (first) { \
+      first = FALSE; \
       value = debug_get_bool_option(name, dfault); \
+   } \
    return value; \
 }
 
@@ -320,8 +322,10 @@ debug_get_option_ ## sufix (void) \
 { \
    static boolean first = TRUE; \
    static long value; \
-   if (first && !(first = FALSE)) \
+   if (first) { \
+      first = FALSE; \
       value = debug_get_num_option(name, dfault); \
+   } \
    return value; \
 }
 
@@ -331,8 +335,10 @@ debug_get_option_ ## sufix (void) \
 { \
    static boolean first = TRUE; \
    static unsigned long value; \
-   if (first && !(first = FALSE)) \
+   if (first) { \
+      first = FALSE; \
       value = debug_get_flags_option(name, flags, dfault); \
+   } \
    return value; \
 }