hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land.
LGTM with one nit. ================ Comment at: include-fixer/SymbolIndexManager.cpp:73 @@ +72,3 @@ + if (IsFullyQualified) + IsMatched &= SymbolContext == Symbol.getContexts().end(); + ---------------- Might be `IsMatched &= (SymbolContext == Symbol.getContexts().end());` for reading clearly? http://reviews.llvm.org/D20424 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits