Re: [PATCH] D13523: ASTMatchers: Keep AllCallbacks in a set instead of a vector

2015-10-07 Thread Daniel Jasper via cfe-commits
djasper closed this revision. djasper added a comment. Submitted as r249598. http://reviews.llvm.org/D13523 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D13523: ASTMatchers: Keep AllCallbacks in a set instead of a vector

2015-10-07 Thread Manuel Klimek via cfe-commits
klimek accepted this revision. klimek added a comment. This revision is now accepted and ready to land. That makes a lot of sense. LG http://reviews.llvm.org/D13523 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bi

[PATCH] D13523: ASTMatchers: Keep AllCallbacks in a set instead of a vector

2015-10-07 Thread Daniel Jasper via cfe-commits
djasper created this revision. djasper added a reviewer: klimek. djasper added a subscriber: cfe-commits. Herald added a subscriber: klimek. AllCallbacks is currently only used to call onStartOfTranslationUnit and onEndOfTranslationUnit on them. In this (and any other scenario I can come up with