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

[Alexander Kornienko via Phabricator via cfe-commits]

alexfh requested changes to this revision.
alexfh added a comment.
This revision now requires changes to proceed.

Apologies again for the long delay.

A diff with full context would still be appreciated. Please read https://llvm.org/docs/Phabricator.html for instructions.

The clang-tools-extra/test/clang-tidy/infrastructure/clang-tidy-diff.cpp test needs to be updated accordingly. See Diff Detail -> Build Status for the build results after uploading a patch to Phabricator.



================
Comment at: clang-tools-extra/clang-tidy/tool/clang-tidy-diff.py:128
+  parser.add_argument('-strip', metavar='NUM', default=0,
+                      help='strip the smallest prefix containing N slashes')
   parser.add_argument('-regex', metavar='PATTERN', default=None,
----------------
s/N/NUM/ ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D49864