22/01/2018 02:54, Neil Horman: > On Sun, Jan 21, 2018 at 09:29:18PM +0100, Thomas Monjalon wrote: > > > $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