alexfh added inline comments.
================
Comment at: clang-tidy/modernize/PassByValueCheck.cpp:87
@@ +86,3 @@
+ /// \brief Whether or not the parameter variable is referred only once in
+ /// the
+ /// given constructor.
----------------
nit: clang-format doesn't properly reflow comments, it just breaks long lines,
so it sometimes needs manual cleanup (or gq in vim).
================
Comment at: clang-tidy/modernize/PassByValueCheck.cpp:134
@@ +133,3 @@
+ : ClangTidyCheck(Name, Context),
+ IncludeStyle(StringSwitch<IncludeSorter::IncludeStyle>(
+ Options.get("IncludeStyle", "llvm") == "llvm" ?
----------------
Does this compile?
http://reviews.llvm.org/D11946
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
