[PATCH] D22129: [clang-rename] add documentation

Ben Jackson via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 11 15:19:01 PDT 2016


puremourning added a subscriber: puremourning.
puremourning added a comment.

Couple of drive-by comments. Thanks for doing this.


================
Comment at: docs/clang-rename.rst:45
@@ +44,3 @@
+
+The tool is currently supporting renaming actions inside single Translation Unit
+only. It is planned to extend tool functionality to support multi-TU renaming
----------------
... currently supports ... inside a single ...

Is more natural English. 

================
Comment at: docs/clang-rename.rst:46
@@ +45,3 @@
+The tool is currently supporting renaming actions inside single Translation Unit
+only. It is planned to extend tool functionality to support multi-TU renaming
+actions in the future.
----------------
... the tool's functionality ...


http://reviews.llvm.org/D22129





More information about the cfe-commits mailing list