[PATCH v3 22/29] patman: Don't ignore lines starting with hash
Simon Glass
sjg at chromium.org
Fri Oct 30 14:46:31 AEDT 2020
These lines can indicate a continuation of an error and should not be
ignored. Fix this.
Fixes: 666eb15e923 ("patman: Handle checkpatch output with notes and code")
Signed-off-by: Simon Glass <sjg at chromium.org>
---
(no changes since v1)
tools/patman/checkpatch.py | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/tools/patman/checkpatch.py b/tools/patman/checkpatch.py
index 263bac3fc90..98d962cd50d 100644
--- a/tools/patman/checkpatch.py
+++ b/tools/patman/checkpatch.py
@@ -95,6 +95,7 @@ def CheckPatch(fname, verbose=False, show_types=False):
re_check = re.compile('CHECK:%s (.*)' % type_name)
re_file = re.compile('#\d+: FILE: ([^:]*):(\d+):')
re_note = re.compile('NOTE: (.*)')
+ re_new_file = re.compile('new file mode .*')
indent = ' ' * 6
for line in result.stdout.splitlines():
if verbose:
@@ -111,8 +112,10 @@ def CheckPatch(fname, verbose=False, show_types=False):
# Skip lines which quote code
if line.startswith(indent):
continue
- # Skip code quotes and #<n>
- if line.startswith('+') or line.startswith('#'):
+ # Skip code quotes
+ if line.startswith('+'):
+ continue
+ if re_new_file.match(line):
continue
match = re_stats_full.match(line)
if not match:
--
2.29.1.341.ge80a0c044ae-goog
More information about the Patchwork
mailing list