Re: [PATCH 1/1] checkpatch: improve operator spacing check

2015-04-06 Thread Joe Perches
On Tue, 2015-04-07 at 13:36 +1000, Sam Bobroff wrote: > Code such as: >x = timercmp(, , <); > Will currently trigger a checkpatch error. e.g. > ERROR: spaces required around that '<' > > This is because the "Ignore operators passed as parameters" check > looks only for a comma following

[PATCH 1/1] checkpatch: improve operator spacing check

2015-04-06 Thread Sam Bobroff
Code such as: x = timercmp(, , <); Will currently trigger a checkpatch error. e.g. ERROR: spaces required around that '<' This is because the "Ignore operators passed as parameters" check looks only for a comma following the operator. Improve the check by also looking for a close

[PATCH 1/1] checkpatch: improve operator spacing check

2015-04-06 Thread Sam Bobroff
Code such as: x = timercmp(now, end, ); Will currently trigger a checkpatch error. e.g. ERROR: spaces required around that '' This is because the Ignore operators passed as parameters check looks only for a comma following the operator. Improve the check by also looking for a close

Re: [PATCH 1/1] checkpatch: improve operator spacing check

2015-04-06 Thread Joe Perches
On Tue, 2015-04-07 at 13:36 +1000, Sam Bobroff wrote: Code such as: x = timercmp(now, end, ); Will currently trigger a checkpatch error. e.g. ERROR: spaces required around that '' This is because the Ignore operators passed as parameters check looks only for a comma following the