diff options
| author | Grazvydas Ignotas <notasas@gmail.com> | 2016-04-16 04:00:10 +0300 |
|---|---|---|
| committer | Ian Romanick <ian.d.romanick@intel.com> | 2016-04-25 12:23:10 +0200 |
| commit | e4fc06a2f8be2de3a08d4adfa4a26df2ae0b5cae (patch) | |
| tree | dd79c0ef687e1189bdfe3ac655e592a95180c48a /src/util | |
| parent | 787a53988cc6bb7a0f2b43c216837d683336b33f (diff) | |
| download | external_mesa3d-e4fc06a2f8be2de3a08d4adfa4a26df2ae0b5cae.zip external_mesa3d-e4fc06a2f8be2de3a08d4adfa4a26df2ae0b5cae.tar.gz external_mesa3d-e4fc06a2f8be2de3a08d4adfa4a26df2ae0b5cae.tar.bz2 | |
util: add MAYBE_UNUSED for config dependent variables
This is mostly for variables that are only used in asserts and cause
unused-but-set-variable warnings in release builds. Could just use
UNUSED directly, but MAYBE_UNUSED should be less confusing and is
similar to what the Linux kernel has.
And yes __attribute__((unused)) can be used on variables on both GCC 4.2
(oldest supported by mesa) and clang 3.0 (just some random old version,
not sure what's the minimum for mesa).
Signed-off-by: Grazvydas Ignotas <notasas@gmail.com>
Reviewed-by: Francisco Jerez <currojerez@riseup.net>
Reviewed-by: Chad Versace <chad.versace@intel.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
Diffstat (limited to 'src/util')
| -rw-r--r-- | src/util/macros.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/util/macros.h b/src/util/macros.h index 0c8958f..f081bb8 100644 --- a/src/util/macros.h +++ b/src/util/macros.h @@ -204,6 +204,8 @@ do { \ #define UNUSED #endif +#define MAYBE_UNUSED UNUSED + #ifdef HAVE_FUNC_ATTRIBUTE_WARN_UNUSED_RESULT #define MUST_CHECK __attribute__((warn_unused_result)) #else |