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