[PATCH] D22805: [clang-include-fixer] Added Emacs integration for clang-include-fixer.

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 26 06:59:46 PDT 2016


hokein added a comment.

Nice! Thanks very much for making emacs integration. I'm not familiar with elisp, but it looks almost good to me. Just a few nits.


================
Comment at: include-fixer/tool/clang-include-fixer.el:8
@@ +7,3 @@
+
+;; This package allows to envoke the 'clnag-include-fixer' within Emacs.
+;; 'clang-include-fixer' provides an automated way of adding #include
----------------
s/envoke/invoke

================
Comment at: include-fixer/tool/clang-include-fixer.el:12
@@ +11,3 @@
+;; <http://clang.llvm.org/extra/include-fixer.html>.
+
+;;; Code:
----------------
Could you add some documents describing how to setup the environment in emacs in `docs/include-fixer.rst`?


https://reviews.llvm.org/D22805





More information about the cfe-commits mailing list