[PATCH] D59734: [clang-tidy] Handle missing yaml module in run-clang-tidy.py

serge via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 25 05:13:18 PDT 2019


serge-sans-paille added inline comments.


================
Comment at: clang-tidy/tool/run-clang-tidy.py:210
+                        help='Create a yaml file to store suggested fixes in, '
+                        'which can be applied with clang-apply-replacements.')
   parser.add_argument('-j', type=int, default=0,
----------------
could be

```
if yaml:
    parser.add_argument('-export-fixes', metavar='filename', dest='export_fixes',
	​                        help='Create a yaml file to store suggested fixes in, '
	​                        'which can be applied with clang-apply-replacements.')
else:
    parser.export_fixes = False
```

This would make later condition easier to read.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59734/new/

https://reviews.llvm.org/D59734





More information about the cfe-commits mailing list