Re: [dpdk-dev] [PATCH] devtools: add long lines to the list of options

2017-03-06 Thread Thomas Monjalon
2017-03-01 12:44, Allain Legacy: > The checkpatch.pl tool is flagging warnings on long debug log strings. > Since splitting these strings makes it difficult to search for logs it is > preferred to allow these as exceptions to the long line rule. The addition > of the LONG_LINE_STRINGS to the list

Re: [dpdk-dev] [PATCH] devtools: add long lines to the list of options

2017-03-02 Thread Gaƫtan Rivet
On Wed, Mar 01, 2017 at 10:08:51AM -0800, Stephen Hemminger wrote: On Wed, 1 Mar 2017 12:44:19 -0500 Allain Legacy wrote: The checkpatch.pl tool is flagging warnings on long debug log strings. Since splitting these strings makes it difficult to search for logs it is preferred to allow these as

Re: [dpdk-dev] [PATCH] devtools: add long lines to the list of options

2017-03-02 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Wednesday, March 1, 2017 6:39 PM > To: Stephen Hemminger > Cc: Legacy, Allain (Wind River) ; > dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] devtools: add long

Re: [dpdk-dev] [PATCH] devtools: add long lines to the list of options

2017-03-01 Thread Thomas Monjalon
2017-03-01 10:08, Stephen Hemminger: > > --- a/devtools/checkpatches.sh > > +++ b/devtools/checkpatches.sh > > @@ -45,7 +45,7 @@ options="$options > > --ignore=LINUX_VERSION_CODE,FILE_PATH_CHANGES,\ > > VOLATILE,PREFER_PACKED,PREFER_ALIGNED,PREFER_PRINTF,\ > > PREFER_KERNEL_TYPES,BIT_MACRO,CONST

Re: [dpdk-dev] [PATCH] devtools: add long lines to the list of options

2017-03-01 Thread Stephen Hemminger
On Wed, 1 Mar 2017 12:44:19 -0500 Allain Legacy wrote: > The checkpatch.pl tool is flagging warnings on long debug log strings. > Since splitting these strings makes it difficult to search for logs it is > preferred to allow these as exceptions to the long line rule. The addition > of the LONG_L

[dpdk-dev] [PATCH] devtools: add long lines to the list of options

2017-03-01 Thread Allain Legacy
The checkpatch.pl tool is flagging warnings on long debug log strings. Since splitting these strings makes it difficult to search for logs it is preferred to allow these as exceptions to the long line rule. The addition of the LONG_LINE_STRINGS to the list of exceptions will allow lines that end w