Re: [PATCH] checkpatch: Allow space leading blank lines in email headers

2017-03-23 Thread Darren Hart
On Wed, Mar 22, 2017 at 11:28:40PM -0700, Joe Perches wrote: > On Wed, 2017-03-22 at 23:20 -0700, Darren Hart wrote: > > I do have an open question regarding how we're going about testing for the > > end > > of the header lines. Since we're not just testing for an empty line to > > separate > >

Re: [PATCH] checkpatch: Allow space leading blank lines in email headers

2017-03-23 Thread Darren Hart
On Wed, Mar 22, 2017 at 11:28:40PM -0700, Joe Perches wrote: > On Wed, 2017-03-22 at 23:20 -0700, Darren Hart wrote: > > I do have an open question regarding how we're going about testing for the > > end > > of the header lines. Since we're not just testing for an empty line to > > separate > >

[PATCH] checkpatch: Allow space leading blank lines in email headers

2017-03-23 Thread Joe Perches
Allow a leading space and otherwise blank link in the email headers as it can be a line wrapped Spamassassin multiple line string or any other valid rfc 2822/5322 email header. The line with space causes checkpatch to erroniously think that it's in the content body, as opposed to headers and thus

[PATCH] checkpatch: Allow space leading blank lines in email headers

2017-03-23 Thread Joe Perches
Allow a leading space and otherwise blank link in the email headers as it can be a line wrapped Spamassassin multiple line string or any other valid rfc 2822/5322 email header. The line with space causes checkpatch to erroniously think that it's in the content body, as opposed to headers and thus

Re: [PATCH] checkpatch: Allow space leading blank lines in email headers

2017-03-23 Thread Joe Perches
On Wed, 2017-03-22 at 23:20 -0700, Darren Hart wrote: > I do have an open question regarding how we're going about testing for the end > of the header lines. Since we're not just testing for an empty line to > separate > headers and body, there is clearly more going on here - but I'm not sure

Re: [PATCH] checkpatch: Allow space leading blank lines in email headers

2017-03-23 Thread Joe Perches
On Wed, 2017-03-22 at 23:20 -0700, Darren Hart wrote: > I do have an open question regarding how we're going about testing for the end > of the header lines. Since we're not just testing for an empty line to > separate > headers and body, there is clearly more going on here - but I'm not sure

Re: [PATCH] checkpatch: Allow space leading blank lines in email headers

2017-03-23 Thread Darren Hart
On Wed, Mar 22, 2017 at 10:35:21PM -0700, Joe Perches wrote: > Allow a leading space and otherwise blank link in the email headers > as it can be a line wrapped Spamassassin multiple line string or any > other valid rfc 2822/5322 email header. > > The line with space causes checkpatch to

Re: [PATCH] checkpatch: Allow space leading blank lines in email headers

2017-03-23 Thread Darren Hart
On Wed, Mar 22, 2017 at 10:35:21PM -0700, Joe Perches wrote: > Allow a leading space and otherwise blank link in the email headers > as it can be a line wrapped Spamassassin multiple line string or any > other valid rfc 2822/5322 email header. > > The line with space causes checkpatch to