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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits