BaseTools/PatchCheck.py: drop redundant line count check

Before doing the subject line length check, the script checks that the
number of lines in the commit message (including subject) is not less
than or equal to zero - and returns if it is.

However, then the test for whether the subject line starts with a CVE
tag inexplicably also checks for whether the number of lines are
greater than or equal to one. This is just clutter, so drop it.

Signed-off-by: Leif Lindholm <leif.lindholm@oss.qualcomm.com>
This commit is contained in:
Leif Lindholm
2025-06-30 15:55:50 +01:00
committed by mergify[bot]
parent 7c40bc06a9
commit 27d44c1abd

View File

@@ -263,7 +263,7 @@ class CommitMessageCheck:
self.error('Empty commit message!')
return
if count >= 1 and re.search(self.cve_re, lines[0]):
if re.search(self.cve_re, lines[0]):
#
# If CVE-xxxx-xxxxx is present in subject line, then limit length of
# subject line to 92 characters