alexfh added a comment.
Actually, I'm not sure why we need groups in that regex. We can instead try
replacing `re.compile('(' + ')|('.join(args.files) + ')')` with
`re.compile('|'.join(args.files))`.
https://reviews.llvm.org/D30607
___
cfe-commits
alexfh added a comment.
https://docs.python.org/2/howto/regex.html#introduction says "The regex module
was removed completely in Python 2.5.". Why would we want to switch to it?
https://reviews.llvm.org/D30607
___
cfe-commits mailing list
cfe-commi
sylvestre.ledru created this revision.
Herald added a subscriber: JDevlieghere.
Avoid limitation of 100 groups for regular expressions
Patch by Manuel Grizonnet
reported here:
https://github.com/llvm-mirror/clang-tools-extra/pull/9
https://reviews.llvm.org/D30607
Files:
clang-tidy/tool/run-c
