diff options
author | Srishti Sharma <srishtishar@gmail.com> | 2017-09-16 02:35:47 +0530 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-09-18 11:52:36 +0200 |
commit | 1521968f3cdfde1df64b68717eb9cfcb9f8fa52f (patch) | |
tree | 78020cd920811873b56d5e04ac557f2aa581a8b8 /drivers/staging/irda/net/irqueue.c | |
parent | 6561e82d93e12744f1fa2048b6a17e5739682749 (diff) |
Staging: irda: Don't use assignment inside if statement
Write assignment statement outside of the if statement. Done
using the following semantic patch by coccinelle.
@@
identifier E;
expression F;
statement S;
@@
-if((E = F))
+E = F;
+if(E)
S
Signed-off-by: Srishti Sharma <srishtishar@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/irda/net/irqueue.c')
-rw-r--r-- | drivers/staging/irda/net/irqueue.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/staging/irda/net/irqueue.c b/drivers/staging/irda/net/irqueue.c index 160dc89335e2..5aab07259ac7 100644 --- a/drivers/staging/irda/net/irqueue.c +++ b/drivers/staging/irda/net/irqueue.c @@ -217,7 +217,8 @@ static __u32 hash( const char* name) while(*name) { h = (h<<4) + *name++; - if ((g = (h & 0xf0000000))) + g = (h & 0xf0000000); + if (g) h ^=g>>24; h &=~g; } |