From a0edf4cb042b72b0d301e5ee6f14a8587e375d53 Mon Sep 17 00:00:00 2001 From: "andrew@webrtc.org" Date: Mon, 5 May 2014 18:18:02 +0000 Subject: [PATCH] Remove ALLOW_UNUSED. Turns out Chromium won't be applying this to COMPILE_ASSERT. We don't need it at all then. R=thakis@chromium.org TBR=thakis@chromium.org Review URL: https://webrtc-codereview.appspot.com/13469005 git-svn-id: http://webrtc.googlecode.com/svn/trunk@6055 4adac7df-926f-26a2-2b94-8c16560cd09d --- webrtc/system_wrappers/interface/compile_assert.h | 2 +- webrtc/typedefs.h | 11 ----------- 2 files changed, 1 insertion(+), 12 deletions(-) diff --git a/webrtc/system_wrappers/interface/compile_assert.h b/webrtc/system_wrappers/interface/compile_assert.h index 46e40ac420..a075184b51 100644 --- a/webrtc/system_wrappers/interface/compile_assert.h +++ b/webrtc/system_wrappers/interface/compile_assert.h @@ -41,7 +41,7 @@ struct CompileAssert { }; #define COMPILE_ASSERT(expr, msg) \ - typedef CompileAssert<(bool(expr))> msg[bool(expr) ? 1 : -1] ALLOW_UNUSED + typedef CompileAssert<(bool(expr))> msg[bool(expr) ? 1 : -1] #endif // __cplusplus >= 201103L #endif // !defined(COMPILE_ASSERT) diff --git a/webrtc/typedefs.h b/webrtc/typedefs.h index d9328c6769..d8977ff4cb 100644 --- a/webrtc/typedefs.h +++ b/webrtc/typedefs.h @@ -96,17 +96,6 @@ typedef unsigned __int64 uint64_t; #define OVERRIDE #endif -// Annotate a variable indicating it's ok if the variable is not used. -// (Typically used to silence a compiler warning when the assignment -// is important for some other reason.) -// Use like: -// int x ALLOW_UNUSED = ...; -#if defined(__GNUC__) -#define ALLOW_UNUSED __attribute__((unused)) -#else -#define ALLOW_UNUSED -#endif - // Annotate a function indicating the caller must examine the return value. // Use like: // int foo() WARN_UNUSED_RESULT;