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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to