[PATCH] D20621: [include-fixer] Create a mode in vim integration to show multiple potential headers.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Mon May 30 08:54:23 PDT 2016


hokein added inline comments.

================
Comment at: include-fixer/IncludeFixer.h:80
@@ +79,3 @@
+    unsigned FirstIncludeOffset=-1U,
+    const clang::format::FormatStyle &Style=clang::format::getLLVMStyle());
+
----------------
Using a default argument in `Style` can simplify the code in some cases, for example in the Unittest. @klimek might have idea? 


http://reviews.llvm.org/D20621





More information about the cfe-commits mailing list