On 03/25/2013 02:56 PM, Stefan Schmidt wrote:
> On Sun, 2013-03-24 at 20:19, Alan Ott wrote:
>> Some of those lines break the 80 column rule. You might get away with
>> it. It's up to you if you want to try.
> If the rule of thumb has not changed lately it depends on how ugly the
> code would look when forcing the 80 char rule. It was always a nice
> point for discussion. :)
>
> I think the MSB and LSB comments can go easily. Breaking up the debug
> print into two lines would be to ugly in my opnion. Maybe I can short
> them. Will think about it before the next submit.
>

It doesn't bother me at all the way they are. Like I've said before
though, I find that patches get through the easiest when nobody
complains, and there's at least one guy who seems to complain about
every formatting issue (at least when I submit). YMMV. It's ultimately
not up to me :) [1]

Alan.

[1] It's good to be able to pass-the-buck :)


------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_mar
_______________________________________________
Linux-zigbee-devel mailing list
Linux-zigbee-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel

Reply via email to