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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
