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

[Haojian Wu via cfe-commits]

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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits