Re: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok

2021-11-25 Thread Thomas Monjalon
22/10/2021 02:11, Jerin Jacob: > On Wed, Oct 20, 2021 at 7:58 PM Andrew Rybchenko > wrote: > > On 10/20/21 5:26 PM, Bruce Richardson wrote: > > > Since we allow line lengths of up to 100, and the CI checkpatches job > > > only check for that amount, the rest of our tooling and docs should > > > re

Re: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok

2021-10-28 Thread Walsh, Conor
> From: dev On Behalf Of Bruce Richardson > Sent: Wednesday 20 October 2021 15:26 > To: dev@dpdk.org > Cc: Andrew Rybchenko ; Richardson, > Bruce > Subject: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 > characters are ok > > Since we allow line lengt

Re: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok

2021-10-25 Thread fengchengwen
On 2021/10/20 22:26, Bruce Richardson wrote: > Since we allow line lengths of up to 100, and the CI checkpatches job > only check for that amount, the rest of our tooling and docs should > reflect this reality. Therefore we can: > > * adjust the editorconfig to use that value, to save editors (e.g

Re: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok

2021-10-22 Thread Xia, Chenbo
> -Original Message- > From: dev On Behalf Of Bruce Richardson > Sent: Wednesday, October 20, 2021 10:26 PM > To: dev@dpdk.org > Cc: Andrew Rybchenko ; Richardson, Bruce > > Subject: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 > characters are ok

Re: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok

2021-10-21 Thread Jerin Jacob
On Wed, Oct 20, 2021 at 7:58 PM Andrew Rybchenko wrote: > > On 10/20/21 5:26 PM, Bruce Richardson wrote: > > Since we allow line lengths of up to 100, and the CI checkpatches job > > only check for that amount, the rest of our tooling and docs should > > reflect this reality. Therefore we can: > >

Re: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok

2021-10-20 Thread Andrew Rybchenko
On 10/20/21 5:26 PM, Bruce Richardson wrote: > Since we allow line lengths of up to 100, and the CI checkpatches job > only check for that amount, the rest of our tooling and docs should > reflect this reality. Therefore we can: > > * adjust the editorconfig to use that value, to save editors (e.g

[dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok

2021-10-20 Thread Bruce Richardson
Since we allow line lengths of up to 100, and the CI checkpatches job only check for that amount, the rest of our tooling and docs should reflect this reality. Therefore we can: * adjust the editorconfig to use that value, to save editors (e.g. vim) from automatically wrapping lines at 80 charac