diff options
| author | 2016-08-26 15:06:11 -0700 | |
|---|---|---|
| committer | 2016-08-26 17:52:41 -0700 | |
| commit | 1a0de6acd03b43a03678b58f47b2f21b5215cc22 (patch) | |
| tree | 7d03f89b9d64ebc40ea382011de524d18a004f73 /compiler/elf_writer_test.cc | |
| parent | cc55a7ce2aa0f10c2da9efaa89855cf075e56e20 (diff) | |
Fix clang-tidy warnings in art.
* Add parentheses around macro parameters, or
use NOLINT to suppress warning.
Bug: 28705665
Test: build with WITH_TIDY=1
Change-Id: Ifc922c2e66215772042bac372754ea70074f0053
Diffstat (limited to 'compiler/elf_writer_test.cc')
| -rw-r--r-- | compiler/elf_writer_test.cc | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/compiler/elf_writer_test.cc b/compiler/elf_writer_test.cc index 449f514184..d5f16637be 100644 --- a/compiler/elf_writer_test.cc +++ b/compiler/elf_writer_test.cc @@ -38,16 +38,16 @@ class ElfWriterTest : public CommonCompilerTest { #define EXPECT_ELF_FILE_ADDRESS(ef, expected_value, symbol_name, build_map) \ do { \ - void* addr = reinterpret_cast<void*>(ef->FindSymbolAddress(SHT_DYNSYM, \ - symbol_name, \ - build_map)); \ + void* addr = reinterpret_cast<void*>((ef)->FindSymbolAddress(SHT_DYNSYM, \ + symbol_name, \ + build_map)); \ EXPECT_NE(nullptr, addr); \ EXPECT_LT(static_cast<uintptr_t>(ART_BASE_ADDRESS), reinterpret_cast<uintptr_t>(addr)); \ - if (expected_value == nullptr) { \ - expected_value = addr; \ + if ((expected_value) == nullptr) { \ + (expected_value) = addr; \ } \ EXPECT_EQ(expected_value, addr); \ - EXPECT_EQ(expected_value, ef->FindDynamicSymbolAddress(symbol_name)); \ + EXPECT_EQ(expected_value, (ef)->FindDynamicSymbolAddress(symbol_name)); \ } while (false) TEST_F(ElfWriterTest, dlsym) { |