Skip to content

Commit b758a87

Browse files
r-barnesfacebook-github-bot
authored andcommitted
Remove unused-variable in ods3/services/storage/gorilla/lib3/gorilla/micro_shard/GmscQuery.h
Summary: LLVM-15 has a warning `-Wunused-variable` which we treat as an error because it's so often diagnostic of a code issue. Unused variables can compromise readability or, worse, performance. This diff either (a) removes an unused variable and, possibly, it's associated code or (b) qualifies the variable with `[[maybe_unused]]`. #buildsonlynotests - Builds are sufficient - If you approve of this diff, please use the "Accept & Ship" button :-) Reviewed By: meyering Differential Revision: D65859992
1 parent f01b20b commit b758a87

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

runtime/platform/assert.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,7 @@
7474
* @param[in] _cond Condition asserted as true.
7575
*/
7676
#define ET_DCHECK(_cond) ((void)0)
77+
#define ET_DEBUG_ONLY [[maybe_unused]]
7778

7879
#else // NDEBUG
7980

@@ -95,6 +96,7 @@
9596
* @param[in] _cond Condition asserted as true.
9697
*/
9798
#define ET_DCHECK(_cond) ET_CHECK(_cond)
99+
#define ET_DEBUG_ONLY
98100

99101
#endif // NDEBUG
100102

0 commit comments

Comments
 (0)