[PATCH] D49864: [clang-tidy] The script clang-tidy-diff.py doesn't accept 'pass by' options (--)

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 24 07:21:36 PDT 2018


alexfh added inline comments.


================
Comment at: clang-tidy/tool/clang-tidy-diff.py:123-130
   if args.fix:
     command.append('-fix')
   if args.checks != '':
     command.append('-checks=' + quote + args.checks + quote)
   if args.quiet:
     command.append('-quiet')
   if args.build_path is not None:
----------------
If we make the script leave out the `--` flag, we should stop forwarding these flags and the `extra_arg(_before)?` below. Otherwise it's too confusing (should one place -fix before `--` or after? what about `-warnings-as-errors`?).

Please also update the usage example at the top.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D49864





More information about the cfe-commits mailing list