Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/15964
Change subject: IMPALA-9761: Fix GCC7 ambiguous else warning for gtest macros ...................................................................... IMPALA-9761: Fix GCC7 ambiguous else warning for gtest macros On GCC7, a dangling-else warning is firing for code like: if (cond1) ASSERT_TRUE(cond2) This is true for several ASSERT_* and EXPECT_* gtest macros. gtest had some code to avoid warnings for code of this form, but that code is no longer effective. gtest now disables the dangling-else warning. Since this is just a matter of adding braces, this adds braces for all those locations. For consistency, this may include locations that were not failing. I found locations by doing: git grep EXPECT_ | grep if git grep ASSERT_ | grep if and manually looking through the output. Testing: - Builds successfully Change-Id: Ieb664afe83736a71508302575e8e66a1b506c985 --- M be/src/exec/parquet/parquet-common-test.cc M be/src/runtime/buffered-tuple-stream-test.cc M be/src/runtime/bufferpool/buffer-pool-test.cc M be/src/runtime/io/disk-io-mgr-test.cc M be/src/runtime/timestamp-test.cc M be/src/testutil/cpu-util.h 6 files changed, 29 insertions(+), 11 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/64/15964/1 -- To view, visit http://gerrit.cloudera.org:8080/15964 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ieb664afe83736a71508302575e8e66a1b506c985 Gerrit-Change-Number: 15964 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <[email protected]>
