[PATCH] D24075: [include-fixer] Support finding headers for the symbol under cursor.

Jens Massberg via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 7 08:50:26 PDT 2016


massberg added a comment.

Added two comments. It took some time for me to recall Emacs Lisps ;)


================
Comment at: include-fixer/tool/clang-include-fixer.el:48
@@ +47,3 @@
+  :group 'clang-include-fixer
+  :type 'bool
+  :risky t)
----------------
Please use 'boolean here

================
Comment at: include-fixer/tool/clang-include-fixer.el:204
@@ -197,3 +203,3 @@
   (message (concat "Calling the include fixer. "
            "This might take some seconds. Please wait."))
 
----------------
Is this message still accurate when using the new query mode?


https://reviews.llvm.org/D24075





More information about the cfe-commits mailing list