Hi,
this removes a semicolon after "do {} while (0)" in several macro bodies
in libsanitizer. This allows the macros to be used in if-then-elses
without curly braces.
Committed as obvious.
Thanks,
- Tom
2017-11-02 Tom de Vries <t...@codesourcery.com>
PR other/82784
* asan/asan_poisoning.cc (CHECK_SMALL_REGION): Remove semicolon
after "do {} while (0)".
* lsan/lsan_common.cc (LOG_POINTERS, LOG_THREADS): Same.
Signed-off-by: Tom de Vries <t...@codesourcery.com>
---
libsanitizer/asan/asan_poisoning.cc | 2 +-
libsanitizer/lsan/lsan_common.cc | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/libsanitizer/asan/asan_poisoning.cc b/libsanitizer/asan/asan_poisoning.cc
index 15cd8ea..1343dfb 100644
--- a/libsanitizer/asan/asan_poisoning.cc
+++ b/libsanitizer/asan/asan_poisoning.cc
@@ -215,7 +215,7 @@ uptr __asan_region_is_poisoned(uptr beg, uptr size) {
uptr __bad = __asan_region_is_poisoned(__p, __size); \
__asan_report_error(pc, bp, sp, __bad, isWrite, __size, 0);\
} \
- } while (false); \
+ } while (false)
extern "C" SANITIZER_INTERFACE_ATTRIBUTE
diff --git a/libsanitizer/lsan/lsan_common.cc b/libsanitizer/lsan/lsan_common.cc
index a3274d5..4afce9d 100644
--- a/libsanitizer/lsan/lsan_common.cc
+++ b/libsanitizer/lsan/lsan_common.cc
@@ -55,12 +55,12 @@ void RegisterLsanFlags(FlagParser *parser, Flags *f) {
#define LOG_POINTERS(...) \
do { \
if (flags()->log_pointers) Report(__VA_ARGS__); \
- } while (0);
+ } while (0)
#define LOG_THREADS(...) \
do { \
if (flags()->log_threads) Report(__VA_ARGS__); \
- } while (0);
+ } while (0)
ALIGNED(64) static char suppression_placeholder[sizeof(SuppressionContext)];
static SuppressionContext *suppression_ctx = nullptr;