aaron.ballman added inline comments.
================ Comment at: clang-tidy/readability/IdentifierNamingCheck.cpp:654 + RangeIsEntirelyWithinMacroArgument || !RangeContainsMacroExpansion; + Failure.ShouldFix = Failure.ShouldFix && RangeCanBeFixed; } ---------------- We could do an early return if `ShouldFix` is already false and simplify this expression with `ShouldFix = RangeCanBeFixed;`, can't we? https://reviews.llvm.org/D25450 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits