alexfh added inline comments. ================ Comment at: clang-tidy/modernize/PassByValueCheck.cpp:134 @@ +133,3 @@ + : ClangTidyCheck(Name, Context), + IncludeStyle(StringSwitch<IncludeSorter::IncludeStyle>( + Options.get("IncludeStyle", "llvm")) ---------------- Thinking a bit more of this, with just two values a conditional operator seems to be better than a StringSwitch.
http://reviews.llvm.org/D11946 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits