Mesa (main): util: remove our redefinition of assert
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Sun Jul 10 01:27:56 UTC 2022
Module: Mesa
Branch: main
Commit: 2f01a22fe47d15c881704380142579ce42f3a5b3
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=2f01a22fe47d15c881704380142579ce42f3a5b3
Author: Marek Olšák <marek.olsak at amd.com>
Date: Thu Jul 7 13:28:27 2022 -0400
util: remove our redefinition of assert
I think we can all agree that our own assert is unnecessary.
Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/6780
Acked-By: Mike Blumenkrantz <michael.blumenkrantz at gmail.com>
Reviewed-by: Emma Anholt <emma at anholt.net>
Acked-by: Rob Clark <robdclark at chromium.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17403>
---
src/gallium/auxiliary/gallivm/lp_bld_swizzle.c | 3 ---
src/mesa/state_tracker/st_texture.c | 3 ---
src/util/u_debug.h | 7 -------
3 files changed, 13 deletions(-)
diff --git a/src/gallium/auxiliary/gallivm/lp_bld_swizzle.c b/src/gallium/auxiliary/gallivm/lp_bld_swizzle.c
index ea2ec780f16..ad03b2c115d 100644
--- a/src/gallium/auxiliary/gallivm/lp_bld_swizzle.c
+++ b/src/gallium/auxiliary/gallivm/lp_bld_swizzle.c
@@ -408,9 +408,6 @@ lp_build_swizzle_aos(struct lp_build_context *bld,
switch (swizzles[i]) {
default:
assert(0);
-#if defined(NDEBUG) || defined(DEBUG)
- FALLTHROUGH;
-#endif
case PIPE_SWIZZLE_X:
case PIPE_SWIZZLE_Y:
case PIPE_SWIZZLE_Z:
diff --git a/src/mesa/state_tracker/st_texture.c b/src/mesa/state_tracker/st_texture.c
index 0e72491e1b7..07ccac68800 100644
--- a/src/mesa/state_tracker/st_texture.c
+++ b/src/mesa/state_tracker/st_texture.c
@@ -187,9 +187,6 @@ st_gl_texture_dims_to_pipe_dims(GLenum texture,
break;
default:
assert(0 && "Unexpected texture in st_gl_texture_dims_to_pipe_dims()");
-#if defined(NDEBUG) || defined(DEBUG)
- FALLTHROUGH;
-#endif
case GL_TEXTURE_3D:
case GL_PROXY_TEXTURE_3D:
*widthOut = widthIn;
diff --git a/src/util/u_debug.h b/src/util/u_debug.h
index b5d9a505cfb..b5c2396896a 100644
--- a/src/util/u_debug.h
+++ b/src/util/u_debug.h
@@ -233,13 +233,6 @@ void _debug_assert_fail(const char *expr,
#endif
-/** Override standard assert macro */
-#ifdef assert
-#undef assert
-#endif
-#define assert(expr) debug_assert(expr)
-
-
/**
* Output the current function name.
*/
More information about the mesa-commit
mailing list