[PATCH] D39332: [clang-refactor] Introduce a new rename rule for qualified symbols

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 7 23:42:03 PST 2017


sammccall accepted this revision.
sammccall added inline comments.


================
Comment at: lib/Tooling/Refactoring/Rename/RenamingAction.cpp:116
+      /*Description=*/
+      R"(Finds and renames qualified symbols in code with no indexer support.
+It is used to move/rename a symbol to a new namespace/name:
----------------
"with no indexer support" isn't meaningful to users I think - "within a translation unit" or "within a source file"?


================
Comment at: lib/Tooling/Refactoring/Rename/RenamingAction.cpp:132
+
+  New code fter rename:
+    namespace foo {
----------------
fter -> after


https://reviews.llvm.org/D39332





More information about the cfe-commits mailing list