Re: [PATCH] D20424: [include-fixer] Make search handle fully qualified names correctly.

2016-05-19 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL270055: [include-fixer] Make search handle fully qualified names correctly. (authored by d0k). Changed prior to commit: http://reviews.llvm.org/D20424?vs=57766&id=57772#toc Repository: rL LLVM http:

Re: [PATCH] D20424: [include-fixer] Make search handle fully qualified names correctly.

2016-05-19 Thread Haojian Wu via cfe-commits
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()

[PATCH] D20424: [include-fixer] Make search handle fully qualified names correctly.

2016-05-19 Thread Benjamin Kramer via cfe-commits
bkramer created this revision. bkramer added reviewers: hokein, ioeric. bkramer added a subscriber: cfe-commits. If a search string starts with "::" we don't want to return any results for suffixes of that string. http://reviews.llvm.org/D20424 Files: include-fixer/SymbolIndexManager.cpp uni