This is an automated email from Gerrit. "Mark Zhuang <mark.zhu...@spacemit.com>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/8365
-- gerrit commit 5a806add5a23853f40c92d3ab6d9c3c99060d14c Author: Mark Zhuang <mark.zhu...@spacemit.com> Date: Fri Oct 25 14:45:06 2024 +0800 checkpatch: check multiple blank lines Enable LINE_SPACING but only check multiple blank lines now Change-Id: I332d4d414a04eec8fc54b49d416a954d30592219 Signed-off-by: Mark Zhuang <mark.zhu...@spacemit.com> diff --git a/.checkpatch.conf b/.checkpatch.conf index 8cb9a3729a..e9ac98ece0 100644 --- a/.checkpatch.conf +++ b/.checkpatch.conf @@ -14,7 +14,6 @@ --ignore ENOSYS --ignore FILE_PATH_CHANGES --ignore GERRIT_CHANGE_ID ---ignore LINE_SPACING --ignore LOGICAL_CONTINUATIONS --ignore MACRO_WITH_FLOW_CONTROL --ignore NEW_TYPEDEFS diff --git a/tools/scripts/checkpatch.pl b/tools/scripts/checkpatch.pl index 01bd547ff5..1f7223b261 100755 --- a/tools/scripts/checkpatch.pl +++ b/tools/scripts/checkpatch.pl @@ -4037,6 +4037,7 @@ sub process { } } +if (!$OpenOCD) { # check for missing blank lines after struct/union declarations # with exceptions for various attributes and macros if ($prevline =~ /^[\+ ]};?\s*$/ && @@ -4056,6 +4057,7 @@ sub process { fix_insert_line($fixlinenr, "\+"); } } +} # !$OpenOCD # check for multiple consecutive blank lines if ($prevline =~ /^[\+ ]\s*$/ && @@ -4070,6 +4072,7 @@ sub process { $last_blank_line = $linenr; } +if (!$OpenOCD) { # check for missing blank lines after declarations # (declarations must have the same indentation and not be at the start of line) if (($prevline =~ /\+(\s+)\S/) && $sline =~ /^\+$1\S/) { @@ -4115,6 +4118,7 @@ sub process { } } } +} # !$OpenOCD # check for spaces at the beginning of a line. # Exceptions: --