[PATCH] D55790: [AST] Add accessSpecs() iterator range for CXXRecordDecl

2018-12-18 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D55790#1333664 , @m4tx wrote: > In D55790#1333627 , @riccibruno > wrote: > > > Do you really have to add an iterator for this particular thing ? > > Why not just use `specific_dec

[PATCH] D55790: [AST] Add accessSpecs() iterator range for CXXRecordDecl

2018-12-17 Thread Mateusz Maćkowski via Phabricator via cfe-commits
m4tx added a comment. In D55790#1333627 , @riccibruno wrote: > Do you really have to add an iterator for this particular thing ? > Why not just use `specific_decl_iterator` in your clang-tidy > checker ? I don't need to - if this is the preferred way t

[PATCH] D55790: [AST] Add accessSpecs() iterator range for CXXRecordDecl

2018-12-17 Thread Bruno Ricci via Phabricator via cfe-commits
riccibruno added a comment. Do you really have to add an iterator for this particular thing ? Why not just use `specific_decl_iterator` in you clang-tidy checker ? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55790/new/ https://reviews.llvm.org/D55790 __

[PATCH] D55790: [AST] Add accessSpecs() iterator range for CXXRecordDecl

2018-12-17 Thread Mateusz Maćkowski via Phabricator via cfe-commits
m4tx created this revision. Herald added a subscriber: cfe-commits. Adds an iterator that allows to loop over C++ class/union/struct access specifier declarations. Repository: rC Clang https://reviews.llvm.org/D55790 Files: include/clang/AST/DeclCXX.h Index: include/clang/AST/DeclCXX.h