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

[serge via Phabricator via cfe-commits]

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



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