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

Benjamin Kramer via cfe-commits cfe-commits at lists.llvm.org
Mon May 30 10:04:36 PDT 2016


bkramer added inline comments.

================
Comment at: include-fixer/IncludeFixer.h:71
@@ +70,3 @@
+/// \param FirstIncludeOffset The insertion point for new include directives.
+/// -1U(UINT_MAX) indicates uninitialization, which will insert the header at
+/// first line. If there is no #include block in the Code, it will create a
----------------
I think you can drop the part about 'uninitialization' from this comment.

================
Comment at: include-fixer/IncludeFixer.h:79
@@ +78,3 @@
+    StringRef Code, StringRef FilePath, StringRef Header,
+    unsigned FirstIncludeOffset=-1U,
+    const clang::format::FormatStyle &Style=clang::format::getLLVMStyle());
----------------
This doesn't look clang-formatted :p


http://reviews.llvm.org/D20621





More information about the cfe-commits mailing list