Mesa (master): util: mark s as MAYBE_UNUSED in _mesa_half_to_unorm8

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Sat Aug 18 00:35:17 UTC 2018


Module: Mesa
Branch: master
Commit: 9387ca29ae465f5e1ee3b4a73c1f7725304bb30e
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9387ca29ae465f5e1ee3b4a73c1f7725304bb30e

Author: Kai Wasserbäch <kai at dev.carbon-project.org>
Date:   Fri Aug 17 16:32:31 2018 +0200

util: mark s as MAYBE_UNUSED in _mesa_half_to_unorm8

Only used, when asserts are enabled.

Fixes an unused-variable warning with gcc-8:
 ../../../src/util/half_float.c: In function '_mesa_half_to_unorm8':
 ../../../src/util/half_float.c:189:14: warning: unused variable 's' [-Wunused-variable]
     const int s = (val >> 15) & 0x1;
               ^

Signed-off-by: Kai Wasserbäch <kai at dev.carbon-project.org>
Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>

---

 src/util/half_float.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/util/half_float.c b/src/util/half_float.c
index 2eff2c84f5..63aec5c5c1 100644
--- a/src/util/half_float.c
+++ b/src/util/half_float.c
@@ -28,6 +28,7 @@
 #include <assert.h>
 #include "half_float.h"
 #include "rounding.h"
+#include "macros.h"
 
 typedef union { float f; int32_t i; uint32_t u; } fi_type;
 
@@ -186,7 +187,7 @@ uint8_t _mesa_half_to_unorm8(uint16_t val)
 {
    const int m = val & 0x3ff;
    const int e = (val >> 10) & 0x1f;
-   const int s = (val >> 15) & 0x1;
+   MAYBE_UNUSED const int s = (val >> 15) & 0x1;
 
    /* v = round_to_nearest(1.mmmmmmmmmm * 2^(e-15) * 255)
     *   = round_to_nearest((1.mmmmmmmmmm * 255) * 2^(e-15))




More information about the mesa-commit mailing list