[PATCH] D23279: clang-reorder-fields

Alexander Shaposhnikov via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 23 17:12:45 PDT 2016


alexshap added inline comments.

================
Comment at: clang-reorder-fields/ReorderFieldsAction.h:30
@@ +29,3 @@
+  std::map<std::string, tooling::Replacements> &Replacements;
+
+public:
----------------
the thing is that
a few weeks ago there was a diff https://reviews.llvm.org/D21748?id=64016.
That diff contains a lot of changes, but what i am focused on - that diff changes the interface of the class RefactoringTool - now the method getReplacements
returns std::map<std::string, Replacements> & (before that it used to be Replacements)
(see include/clang/Tooling/Refactoring.h the line 48)
All the tools (like clang-rename etc) have been adjusted to consume this map.
That's why i am using it as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D23279





More information about the cfe-commits mailing list