Avoid limitation of 100 groups for regular expressions
Patch by Manuel Grizonnet
reported here:
https://github.com/llvm-mirror/clang-tools-extra/pull/9
Differential D30607
Replace re module by regex module in run-clang-tidy script sylvestre.ledru on Mar 4 2017, 6:22 AM. Authored by
Details
Avoid limitation of 100 groups for regular expressions reported here:
Diff Detail Event TimelineComment Actions 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? Comment Actions 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)). |