Re: [PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-06 Thread Joe Perches
On Tue, 2013-08-06 at 11:05 +0300, Phil Carmody wrote: > > On Mon, 2013-08-05 at 14:08 +0300, Phil Carmody wrote: > > > Lines with incorrect spacing around an operator, such as: > > > bystander, correct,incorrect > > > would get "fixed" to > > > bystander,correct, incorrect > > > as the

RE: [PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-06 Thread Phil Carmody
> On Mon, 2013-08-05 at 14:08 +0300, Phil Carmody wrote: > > Lines with incorrect spacing around an operator, such as: > > bystander, correct,incorrect > > would get "fixed" to > > bystander,correct, incorrect > > as the correct argument as well as the incorrectly-spaced operator > > were both

RE: [PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-06 Thread Phil Carmody
On Mon, 2013-08-05 at 14:08 +0300, Phil Carmody wrote: Lines with incorrect spacing around an operator, such as: bystander, correct,incorrect would get fixed to bystander,correct, incorrect as the correct argument as well as the incorrectly-spaced operator were both being trimmed.

Re: [PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-06 Thread Joe Perches
On Tue, 2013-08-06 at 11:05 +0300, Phil Carmody wrote: On Mon, 2013-08-05 at 14:08 +0300, Phil Carmody wrote: Lines with incorrect spacing around an operator, such as: bystander, correct,incorrect would get fixed to bystander,correct, incorrect as the correct argument as well

Re: [PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-05 Thread Joe Perches
On Mon, 2013-08-05 at 14:08 +0300, Phil Carmody wrote: > Lines with incorrect spacing around an operator, such as: > bystander, correct,incorrect > would get "fixed" to > bystander,correct, incorrect > as the correct argument as well as the incorrectly-spaced operator > were both being

[PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-05 Thread Phil Carmody
Lines with incorrect spacing around an operator, such as: bystander, correct,incorrect would get "fixed" to bystander,correct, incorrect as the correct argument as well as the incorrectly-spaced operator were both being trimmed. The correct argument only needs to be right trimmed.

[PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-05 Thread Phil Carmody
Lines with incorrect spacing around an operator, such as: bystander, correct,incorrect would get fixed to bystander,correct, incorrect as the correct argument as well as the incorrectly-spaced operator were both being trimmed. The correct argument only needs to be right trimmed.

Re: [PATCH 1/1] checkpatch: fix some whitespace issues caused by --fix

2013-08-05 Thread Joe Perches
On Mon, 2013-08-05 at 14:08 +0300, Phil Carmody wrote: Lines with incorrect spacing around an operator, such as: bystander, correct,incorrect would get fixed to bystander,correct, incorrect as the correct argument as well as the incorrectly-spaced operator were both being trimmed. The