commit e6f3d66bae371ddf8ccd91d26ec788dd75d3f5a7
author Peter Kasting <pkasting@chromium.org> Mon Nov 07 22:57:33 2022 +0000
committer LUCI CQ <infra-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Nov 07 22:57:33 2022 +0000
tree 668334596882f749682af940d50260e1f2adb0e8
parent d9db3f6fd8ec38121d5255b8fbded901e7ca16eb

Fix bad regex.

This corrects an oversight in
https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/4004597
that led to IWYU zealously over-recommending #includes.

Bug: chromium:1368812
Change-Id: Ic4b2f93271206be79371378038bdedd40f7079ed
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/4010619
Auto-Submit: Peter Kasting <pkasting@chromium.org>
Reviewed-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>

cpplint.py

diff --git a/cpplint.py b/cpplint.py
index 4ce789e..209c3ac 100755
--- a/cpplint.py
+++ b/cpplint.py
@@ -5350,7 +5350,7 @@
   for _template in _templates:
     # Match max<type>(..., ...), max(..., ...), but not foo->max or foo.max.
     _re_pattern_headers_maybe_templates.append(
-        (re.compile(r'(?<![>.]\b)' + _template + r'(<.*?>)?\([^\)]'), _template,
+        (re.compile(r'(?<![>.])\b' + _template + r'(<.*?>)?\([^\)]'), _template,
          _header))
 
 # Other scripts may reach in and modify this pattern.