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

Benjamin Kramer via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 11 08:02:23 PDT 2016


bkramer added inline comments.

================
Comment at: docs/clang-rename.rst:17
@@ +16,3 @@
+
+The tool development is in very early stage, so you might encounter bugs and
+crashes. Submitting reports with information about how to reproduce the issue
----------------
maybe 'is in a very early development stage"?

================
Comment at: docs/clang-rename.rst:19
@@ +18,3 @@
+crashes. Submitting reports with information about how to reproduce the issue
+to `llvm bugtracker <https://llvm.org/bugs>`_ will definitely help the project.
+If you have any ideas or suggestions, you might want to put a feature request
----------------
to the LLVM bug tracker

================
Comment at: docs/clang-rename.rst:50
@@ +49,3 @@
+:program:`clang-rename` also aims to be easily integrated into popular text
+editors, such as Vim, and improve workflow of users.
+
----------------
... the workflow ...

================
Comment at: docs/clang-rename.rst:52
@@ +51,3 @@
+
+Although command line interface exists, it is highly recommended to use text
+editors interface instead for better experience.
----------------
... a command line interface exists, it is highly recommended to use the text editor interface ...


http://reviews.llvm.org/D22129





More information about the cfe-commits mailing list