[PATCH] D24380: [migrate-tool] Framework for a codebase-dependent migration tool.

Eugene Zelenko via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 9 10:15:21 PDT 2016


Eugene.Zelenko added a subscriber: Eugene.Zelenko.
Eugene.Zelenko added a comment.

I think will be good idea to await clang-refactor and merge code there.

Please run Include What You Use. Code use a lot of containers and will be good to include them explicitly.


================
Comment at: migrate-tool/HeaderBuild.cpp:28
@@ +27,3 @@
+}
+class Namespace {
+public:
----------------
Please separate with empty line.


https://reviews.llvm.org/D24380





More information about the cfe-commits mailing list