22/01/2018 02:54, Neil Horman:

On Sun, Jan 21, 2018 at 09:29:18PM +0100, Thomas Monjalon wrote:

quoted

quoted

 	$verbose || printf '\n### %s\n\n' "$3"
 	printf '%s\n' "$report" | sed -n '1,/^total:.*lines checked$/p'
+
+	echo
+	echo "Checking API additions/removals:"

You should respect $verbose before printing such header.

I can add a quiet/verbose mode option, but I didn't think it was needed here
since its being run automatically from within checkpatches.

I mean there is a verbose option already.
So you just have to take it into account when printing.
Thanks