[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