Re: [PATCH] D21175: [include-fixer] do not index friend function declaration.

2016-06-09 Thread Eric Liu via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL272261: [include-fixer] do not index friend function declaration. (authored by ioeric). Changed prior to commit: http://reviews.llvm.org/D21175?vs=60147&id=60168#toc Repository: rL LLVM http://revie

Re: [PATCH] D21175: [include-fixer] do not index friend function declaration.

2016-06-09 Thread Benjamin Kramer via cfe-commits
bkramer accepted this revision. bkramer added a comment. This revision is now accepted and ready to land. Makes sense. http://reviews.llvm.org/D21175 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listi

[PATCH] D21175: [include-fixer] do not index friend function declaration.

2016-06-09 Thread Eric Liu via cfe-commits
ioeric created this revision. ioeric added a reviewer: bkramer. ioeric added a subscriber: cfe-commits. we want to exclude friend declaration, but the `DeclContext` of a friend function declaration is not the class in which it is declared, so we need to explicitly check if the parent is a `friendD