[dpdk-dev] [PATCH v2] checkpatches.sh: Add checks for ABI symbol addition
Thomas Monjalon
thomas at monjalon.net
Mon Jan 22 03:05:36 CET 2018
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
More information about the dev
mailing list