[PATCH] D22567: [include-fixer] Add mising qualifiers to all instances of an unidentified symbol.
    Haojian Wu via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Thu Jul 21 03:09:44 PDT 2016
    
    
  
hokein added inline comments.
================
Comment at: include-fixer/tool/ClangIncludeFixer.cpp:365
@@ +364,3 @@
+  for (const auto &Info : Context.getQuerySymbolInfos()) {
+    Replacements->insert({FilePath, Info.Range.getOffset(),
+                          Info.Range.getLength(),
----------------
ioeric wrote:
> Now that we insert namespace qualifiers, we might want to do a `formatReplacements` before applying them.
Good point! So that we won't break the current code style. The line probably exceeds 80 characters after adding qualifiers.
https://reviews.llvm.org/D22567
    
    
More information about the cfe-commits
mailing list