[PATCH] D35012: [refactor] Add the AST source selection component

[Manuel Klimek via Phabricator via cfe-commits]

klimek added a comment.

Apart from nits, looks OK to me - Eric, are all your concerns addressed?



================
Comment at: include/clang/Tooling/Refactoring/ASTSelection.h:51-53
+  ast_type_traits::DynTypedNode Node;
+  SourceSelectionKind SelectionKind;
+  std::vector<SelectedASTNode> Children;
----------------
As this is a public interface, perhaps we should make them const?


Repository:
  rL LLVM

https://reviews.llvm.org/D35012