[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
Wed May 25 10:12:06 PDT 2016
bkramer added inline comments.
================
Comment at: include-fixer/IncludeFixer.h:51
@@ -50,6 +50,3 @@
- /// Headers to be added.
- std::set<std::string> &Headers;
-
- /// Replacements are written here.
- std::vector<clang::tooling::Replacement> &Replacements;
+ /// The context that contains all information about queried symbol.
+ IncludeFixerContext &Context;
----------------
maybe 'about the symbol being queried'?
================
Comment at: include-fixer/IncludeFixer.h:72
@@ +71,3 @@
+ StringRef Code, StringRef FilePath, StringRef FallbackStyle,
+ unsigned FirstIncludeOffset, const std::string &Header);
+
----------------
'Header' looks like it should be a StringRef too.
================
Comment at: include-fixer/IncludeFixerContext.h:19
@@ +18,3 @@
+
+struct IncludeFixerContext {
+ std::string SymbolIdentifer;
----------------
I think some doxygen comments would be in order for this struct and its members.
================
Comment at: include-fixer/tool/ClangIncludeFixer.cpp:62
@@ -58,1 +61,3 @@
+cl::opt<bool> OuputHeaders(
+ "output-headers",
----------------
typo: Ouput
http://reviews.llvm.org/D20621
More information about the cfe-commits
mailing list