[PATCH] D20827: [include-fixer] Use YAML format in -output-headers and -insert-header mode.
Benjamin Kramer via cfe-commits
cfe-commits at lists.llvm.org
Tue May 31 11:45:29 PDT 2016
bkramer added inline comments.
================
Comment at: include-fixer/tool/ClangIncludeFixer.cpp:80
@@ +79,3 @@
+ cl::desc("Print the symbol being queried and all its relevant headers in\n"
+ "the YAML format to stdout:\n"
+ " ---\n"
----------------
drop 'the'
================
Comment at: include-fixer/tool/clang-include-fixer.py:22
@@ -21,2 +21,3 @@
import vim
+import yaml
----------------
We cannot rely on python-yaml being available, I don't want users to install python modules just to run include-fixer. Can we restrict ourselves to the JSON subset of YAML here?
http://reviews.llvm.org/D20827
More information about the cfe-commits
mailing list