Re: [PATCH] D20960: [include-fixer] Be smarter about inserting symbols for a prefix.

2016-06-03 Thread Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL271671: [include-fixer] Be smarter about inserting symbols for a prefix. (authored by d0k). Changed prior to commit: http://reviews.llvm.org/D20960?vs=59545=59550#toc Repository: rL LLVM

Re: [PATCH] D20960: [include-fixer] Be smarter about inserting symbols for a prefix.

2016-06-03 Thread Eric Liu via cfe-commits
ioeric accepted this revision. ioeric added a comment. This revision is now accepted and ready to land. LG. http://reviews.llvm.org/D20960 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D20960: [include-fixer] Be smarter about inserting symbols for a prefix.

2016-06-03 Thread Benjamin Kramer via cfe-commits
bkramer created this revision. bkramer added reviewers: hokein, ioeric. bkramer added a subscriber: cfe-commits. If prefix search finds something where nothing can be nested under (e.g. a variable or macro) don't add it to the result. This is for cases like: header.h: extern int a; file.cc: