lkml.org 
[lkml]   [2022]   [Apr]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] staging: rtl8712: remove Unnecessary parentheses
From
Date
On Wed, 2022-04-06 at 17:37 +0200, Julia Lawall wrote:
> If you want to capitalize something in the subject, it could be the first
> word. Capitalizing the second word is a bit strange.
>
> On Wed, 6 Apr 2022, Alaa Mohamed wrote:
>
> > Reported by checkpatch:
> >
> > CHECK: Unnecessary parentheses
>
> Indicating that the problem was detected by checkpatch is good. But
> actually, the parentheses all have the same property. So you could use
> the log message to describe what kind of unnecessary parentheses were
> removed. That would help the maintainer know what to look for.
[]

> > diff --git a/drivers/staging/rtl8712/rtl871x_ioctl_linux.c b/drivers/staging/rtl8712/rtl871x_ioctl_linux.c
[]
> > @@ -82,9 +82,9 @@ static inline void handle_pairwise_key(struct sta_info *psta,
> > (param->u.crypt. key_len > 16 ? 16 : param->u.crypt.key_len));
> > if (strcmp(param->u.crypt.alg, "TKIP") == 0) { /* set mic key */
> > memcpy(psta->tkiptxmickey. skey,

Also, the space after the periods here are not great either.

memcpy(psta->tkiptxmickey. skey,
vs
memcpy(psta->tkiptxmickey.skey,


\
 
 \ /
  Last update: 2022-04-06 21:16    [W:0.055 / U:0.124 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site