================ Comment at: clang-tidy/tool/ClangTidyMain.cpp:295-299 @@ +294,7 @@ + + const bool DisableFixes = Fix && FoundErrors && !FixErrors; + if (DisableFixes) + Fix = false; + else if (FixErrors) + Fix = true; + ---------------- It seems strange to first have to compute DisableFixes based on FixErrors, and then use FixErrors again. If we want FixErrors to imply Fix, why not have the if (FixErrors) Fix = true; before the whole block?
http://reviews.llvm.org/D6059 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
