lkml.org 
[lkml]   [2012]   [Jan]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] Staging; bcm/CmHost.c: Style cleanup
On Wed, 25 Jan 2012, Dan Carpenter wrote:

> This is probably going to need to be redone on top of the other bcm
> cleanup patches anyway.
>
If that turns out to be so, well, so be it. I was not aware of other
patches, so my bad...

> On Tue, Jan 24, 2012 at 11:46:31PM +0100, Jesper Juhl wrote:
> > + pstClassifierEntry->ucDestPortRangeLength
> > + = psfCSType->cCPacketClassificationRule.u8ProtocolDestPortRangeLength/4;
>
> Put the equals on the first line when you break something up like
> this. Also the real fix here is to choose shorter and better names
> than "cCPacketClassificationRule.u8ProtocolDestPortRangeLength".
>
> If you're breaking up a condition normally the && or || go on the
> first line as well.
> if (foo &&
> bar &&
> baz) { ...
>
> Although that's not in CodingStyle so it's not a "redo the patch"
> situation.
>
As you say, "not in CodingStyle", so I guess it's down to your personal
preference vs my personal preference vs what's most common in the kernel
vs maintainers preference.

I did it the way I did for a reason.
I personally find that if one puts the '=', '==', '&&', '||', '+',
<whatever> at the end of each line on a multi-line statement, then they
are easy to overlook when just reading the code casually. If, however, one
puts them at the start of each line, they stand out and are easier to see
(harder to miss). That's why I did as I did. You may not agree, but at
least now you know my reason for doing it the way I did.


--
Jesper Juhl <jj@chaosbits.net> http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.



\
 
 \ /
  Last update: 2012-01-28 00:45    [W:0.053 / U:0.072 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site