From: Alexander Smirnov <alex.bluesman.smir...@gmail.com>
Date: Mon, 19 Dec 2011 19:33:50 +0300

>       int rc = -ENOBUFS;
>       struct net_device *dev;
> +        int type = -EINVAL;

Bad indentation.

Or do you mean to tell me this doesn't look like total garbage and
stand out like a sore thumb on your computer screen?

> +             if (type > __IEEE802154_DEV_MAX) {
> +                     return -EINVAL;
> +             }

No need for openning and closing braces here, there is simply one
statement in the basic block.

If you'll need them later, add them later, not now.

------------------------------------------------------------------------------
Learn Windows Azure Live!  Tuesday, Dec 13, 2011
Microsoft is holding a special Learn Windows Azure training event for 
developers. It will provide a great way to learn Windows Azure and what it 
provides. You can attend the event by watching it streamed LIVE online.  
Learn more at http://p.sf.net/sfu/ms-windowsazure
_______________________________________________
Linux-zigbee-devel mailing list
Linux-zigbee-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel

Reply via email to