janosimas requested review of this revision.
janosimas 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:
----------------
alexfh wrote:
> 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.
What about keep the current `--` behavior and add a new flag
`-extra-tidy-flags` ?
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D49864
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
