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

Kirill Bobyrev via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 5 05:19:56 PDT 2016


omtcyfz added inline comments.

================
Comment at: change-namespace/ChangeNamespace.cpp:480
@@ +479,3 @@
+    Replaces = Replaces.merge(NewReplacements);
+    format::FormatStyle Style = format::getStyle("file", FilePath, "google");
+    // Clean up old namespaces if there is nothing in it after moving.
----------------
omtcyfz wrote:
> By the way, shouldn't this one be "LLVM"?
Alternatively, it might be nice to provide an option to specify desired formatting style.


https://reviews.llvm.org/D24183





More information about the cfe-commits mailing list