When checking a valid patch from standard input, the footer lines of the report are not filtered out.
The function check is called outside of any loop, so the statement continue has no effect and the footer is printed. Fixes: 8005feef421d ("scripts: add standard input to checkpatch") Signed-off-by: Thomas Monjalon <thomas.monjalon at 6wind.com> --- scripts/checkpatches.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/checkpatches.sh b/scripts/checkpatches.sh index 336cc7b..cfe262b 100755 --- a/scripts/checkpatches.sh +++ b/scripts/checkpatches.sh @@ -94,7 +94,7 @@ check () { # <patch> <commit> <title> else report=$($DPDK_CHECKPATCH_PATH $options - 2>/dev/null) fi - [ $? -ne 0 ] || continue + [ $? -ne 0 ] || return 0 $verbose || printf '\n### %s\n\n' "$3" printf '%s\n' "$report" | sed -n '1,/^total:.*lines checked$/p' status=$(($status + 1)) -- 2.7.0