linux-kernel - Re: [PATCH 2/3] staging: wlags49_h2: wl_if.h: fixes several coding style issues

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CALqZ13v31eMRhXQ_12a9mYnF1L6gEzM8TqE9_zUUHQ_RLToAKg@mail.gmail.com>
Date:	Tue, 20 Nov 2012 15:00:01 +0200
From:	Johan Meiring <johanmeiring@...il.com>
To:	Dan Carpenter <dan.carpenter@...cle.com>
Cc:	pe1dnn@...at.org, jkosina@...e.cz, standby24x7@...il.com,
	viro@...iv.linux.org.uk, gregkh@...uxfoundation.org,
	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/3] staging: wlags49_h2: wl_if.h: fixes several coding
 style issues

100%, will do.

On 19 November 2012 09:17, Dan Carpenter <dan.carpenter@...cle.com> wrote:
> On Fri, Nov 16, 2012 at 06:54:43PM +0200, Johan Meiring wrote:
>> This commit sorts out several coding style issues, including the use of C99 // comments, spaces at the start of lines, spaces before tabs, incorrect brace placement, and macros with complex values that have not been wrapped in parentheses.  The only remaining issue is the use of typedefs in two instances, but that's something that the original author might need to have a look at.
>>
>
> Could you line wrap your commit message at something like 72-73
> characters?  I think that's they rule on email, but I forget.
>
> Normally we ask that people break these kinds of changes into
> multiple patches.
> [patch 1/x] Staging: wlags49_h2: wl_if.h: use tab indents
> [patch 2/x] Staging: wlags49_h2: wl_if.h: fix comment style
> etc...
>
>> +/*
>> +#define HCF_MIN_COMM_QUALITY        0
>> +#define HCF_MAX_COMM_QUALITY        92
>> +#define HCF_MIN_SIGNAL_LEVEL        47
>> +#define HCF_MAX_SIGNAL_LEVEL        138
>> +#define HCF_MIN_NOISE_LEVEL         47
>> +#define HCF_MAX_NOISE_LEVEL         138
>> +#define HCF_0DBM_OFFSET             149
>> +*/
>
> Just delete this dead code.
>
>> +#define SIOCSIWNETNAME              (SIOCDEVPRIVATE+1)
>> +#define SIOCGIWNETNAME              (SIOCDEVPRIVATE+2)
>> +#define SIOCSIWSTANAME              (SIOCDEVPRIVATE+3)
>> +#define SIOCGIWSTANAME              (SIOCDEVPRIVATE+4)
>> +#define SIOCSIWPORTTYPE             (SIOCDEVPRIVATE+5)
>> +#define SIOCGIWPORTTYPE             (SIOCDEVPRIVATE+6)
>
> Could you put spaces around the '+' operator?
> #define SIOCSIWNETNAME          (SIOCDEVPRIVATE + 1)
>
> regards,
> dan carpenter
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/