[llvm] [NFC] Fix dangling-else warning (PR #112817)
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 18 06:01:29 PDT 2024
teresajohnson wrote:
> Given that omit braces in such cases is part of LLVM's style guide
Reference: https://llvm.org/docs/CodingStandards.html#don-t-use-braces-on-simple-single-statement-bodies-of-if-else-loop-statements
> I wonder whether it may be better to use `-Wno-dangling-else` when compiling gtest based code using GCC?
>
> As Clang doesn't have this false positive, this warning will creep back in over time.
Agree. But where is the else in this case? I assume it is inside the gtest macros being invoked in the if statement bodies - might be better to add fixes there?
https://github.com/llvm/llvm-project/pull/112817
More information about the llvm-commits
mailing list