Re: [PATCH] scsi: remove extra white space at the end of the line

2018-01-22 Thread Xose Vazquez Perez
Jason Yan wrote: > My editor always try to remove the extra white space at the end of the > line when I make some changes. I'm tired of adjusting them manually. A real (trailing) clean up is done with: find -type f -exec sed -i 's/[[:cntrl:]]\+$//' {} + Trailing spaces, tabs and ^L were deleted

Re: [PATCH] scsi: remove extra white space at the end of the line

2018-01-08 Thread Johannes Thumshirn
"Martin K. Petersen" writes: > I am thoroughly annoyed by all the legacy whitespace problems. I've been > working on two different sd patch series over the holidays and both > caused me no end of grief due to legacy formatting issues. > > I have had an unbreak-sd patch

Re: [PATCH] scsi: remove extra white space at the end of the line

2018-01-03 Thread Martin K. Petersen
Johannes, > All in all I'm so much in favour of this cleanup patch (and even would > apprechiate more of these, especailly getting rid of all those > camelCase foo we have in the scsi hba drivers). > > Let's see how Martin and James deceide about it. I am thoroughly annoyed by all the legacy

Re: [PATCH] scsi: remove extra white space at the end of the line

2017-12-23 Thread Finn Thain
Hi Bart, On Thu, 21 Dec 2017, Bart Van Assche wrote: > There are several reasons why most kernel maintainers ignore patches > like this one silently: > I don't disagree with your conclusion but I think that these objections may have solutions, at least in principle. Your message prompted me

Re: [PATCH] scsi: remove extra white space at the end of the line

2017-12-21 Thread Johannes Thumshirn
On Thu, Dec 21, 2017 at 04:09:07PM +, Bart Van Assche wrote: > On Thu, 2017-12-21 at 10:40 +0800, Jason Yan wrote: > > My editor always try to remove the extra white space at the end of the > > line when I make some changes. I'm tired of adjusting them manually. > > Can we remove them in

Re: [PATCH] scsi: remove extra white space at the end of the line

2017-12-21 Thread Jason Yan
On 2017/12/22 0:09, Bart Van Assche wrote: On Thu, 2017-12-21 at 10:40 +0800, Jason Yan wrote: My editor always try to remove the extra white space at the end of the line when I make some changes. I'm tired of adjusting them manually. Can we remove them in mainline? Hello Jason, I think

Re: [PATCH] scsi: remove extra white space at the end of the line

2017-12-21 Thread Bart Van Assche
On Thu, 2017-12-21 at 10:40 +0800, Jason Yan wrote: > My editor always try to remove the extra white space at the end of the > line when I make some changes. I'm tired of adjusting them manually. > Can we remove them in mainline? Hello Jason, I think this means that your editor has been

Re: [PATCH] scsi: remove extra white space at the end of the line

2017-12-21 Thread Johannes Thumshirn
On Thu, Dec 21, 2017 at 05:41:00AM +, Damien Le Moal wrote: > Jason, > > On 12/21/17 11:40, Jason Yan wrote: > > My editor always try to remove the extra white space at the end of the > > line when I make some changes. I'm tired of adjusting them manually. > > Can we remove them in mainline?

Re: [PATCH] scsi: remove extra white space at the end of the line

2017-12-20 Thread Damien Le Moal
Jason, On 12/21/17 11:40, Jason Yan wrote: > My editor always try to remove the extra white space at the end of the > line when I make some changes. I'm tired of adjusting them manually. > Can we remove them in mainline? > > Signed-off-by: Jason Yan > --- >

[PATCH] scsi: remove extra white space at the end of the line

2017-12-20 Thread Jason Yan
My editor always try to remove the extra white space at the end of the line when I make some changes. I'm tired of adjusting them manually. Can we remove them in mainline? Signed-off-by: Jason Yan --- drivers/scsi/scsi.c | 2 +- drivers/scsi/scsi_lib.c | 20