[PATCH] D24183: A clang tool for changing surrouding namespaces of class/function definitions.

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 9 02:39:22 PDT 2016


omtcyfz added inline comments.

================
Comment at: change-namespace/ChangeNamespace.cpp:232
@@ +231,3 @@
+}
+
+// FIXME(ioeric): handle the following symbols:
----------------
handle == "ioeric" here

s/FIXME(ioeric)/FIXME

================
Comment at: change-namespace/ChangeNamespace.cpp:336
@@ +335,3 @@
+// creates an `InsertForwardDeclaration` to insert the forward declaration back
+// into the old namespace after moving code from the old namespace to the new
+// namespace.
----------------
Ah, I see. Thank you!


https://reviews.llvm.org/D24183





More information about the cfe-commits mailing list