lkml.org 
[lkml]   [2012]   [Dec]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/3] wlan-ng/prism2mgmt.c: added parenthesis to macro
On Mon, Dec 17, 2012 at 02:38:01PM +0100, Sebastian Wankerl wrote:
> From: Sebastian Wankerl <sisewank@faui49man10.informatik.uni-erlangen.de>
>
> Signed-off-by: Sebastian Wankerl <sisewank@cip.cs.fau.de>
> Signed-off-by: Sebastian Ehrenfels <qi50dube@cip.cs.fau.de>
>
> ---
> drivers/staging/wlan-ng/prism2mgmt.c | 25 +++++++++++++++----------
> 1 file changed, 15 insertions(+), 10 deletions(-)
>
> diff --git a/drivers/staging/wlan-ng/prism2mgmt.c b/drivers/staging/wlan-ng/prism2mgmt.c
> index 4efa9bc..62b94b0 100644
> --- a/drivers/staging/wlan-ng/prism2mgmt.c
> +++ b/drivers/staging/wlan-ng/prism2mgmt.c
> @@ -415,11 +415,14 @@ int prism2mgmt_scan_results(wlandevice_t *wlandev, void *msgp)
> break;
>
> #define REQBASICRATE(N) \
> - if ((count >= N) && DOT11_RATE5_ISBASIC_GET(item->supprates[(N)-1])) { \
> - req->basicrate ## N .data = item->supprates[(N)-1]; \
> - req->basicrate ## N .status = \
> - P80211ENUM_msgitem_status_data_ok; \
> - }
> + do { \
> + if ((count >= N) && DOT11_RATE5_ISBASIC_GET( \
> + item->supprates[(N)-1])) { \

This is a very confusing way to indent. The proper way is:

do { \
if (count >= N && \
DOT11_RATE5_ISBASIC_GET(item->supprates[(N)-1])) { \

The "DOT11_RATE5_ISBASIC_GET(item->supprates[(N)-1])" call should be
on one line if possible.

The start 'c' in count and the 'D' in DOT11_ should be in the same
column. You will need to add some space characters to make this
work.

> + req->basicrate ## N .data = item->supprates[(N)-1]; \
> + req->basicrate ## N .status = \
> + P80211ENUM_msgitem_status_data_ok; \
> + } \

regards,
dan carpenter


\
 
 \ /
  Last update: 2012-12-17 16:21    [W:0.082 / U:0.908 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site