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

Reply via email to