Re: [PATCH 03/10] net: add IEEE 802.15.4 socket familyimplementation
From: Johannes Berg
Date: Thu Jun 04 2009 - 10:15:56 EST
On Thu, 2009-06-04 at 18:10 +0400, Dmitry Eremin-Solenikov wrote:
> We do try to fit the lines into 80 chars, but when that seems logical.
> I strongly dislike breaking comparations between lines.
There are a whole bunch of improvements possible at this particular
spot:
switch (X) {
case Y:
if (A) {
...
}
break;
...
}
can be written as
switch (X) {
case Y:
if (!A)
break;
...
break;
}
and
for (...) {
if (A) {
...
break;
}
}
can be written as
for (...) {
if (!A)
continue;
...
break;
}
etc.
Show some creativity!
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part