[PATCH 2/6] staging: speakup: Avoid multiple assignments on same line

[PATCH 0/6] multiple checkpatch issues · Arushi Singhal <hidden> · 2017-03-04
[PATCH 2/6] staging: speakup: Avoid multiple assignments on same line · Arushi Singhal <hidden> · 2017-03-04
[PATCH 3/6] staging: speakup: identation should use tabs · Arushi Singhal <hidden> · 2017-03-04
[PATCH 1/6] staging: speakup: fixes braces {} should be used on all arms of this statement · Arushi Singhal <hidden> · 2017-03-04
[PATCH 6/6] staging: wilc1000: Logical continuations should be on the previous line · Arushi Singhal <hidden> · 2017-03-04
[PATCH 5/6] staging: wilc1000: function prototype argument should have identifier name · Arushi Singhal <hidden> · 2017-03-04
[PATCH 4/6] staging: wilc1000: Alignment should match open parenthesis · Arushi Singhal <hidden> · 2017-03-04

From: Arushi Singhal <hidden>
Date: 2017-03-04 15:58:17
Also in: lkml
Subsystem: staging subsystem, the rest · Maintainers: Greg Kroah-Hartman, Linus Torvalds

This patch fixes the checkpatch.pl warning "multiple assignments
should be avoided."

Signed-off-by: Arushi Singhal <redacted>
---
 drivers/staging/speakup/main.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/speakup/main.c b/drivers/staging/speakup/main.c
index 47cea629ba0b..604335390b87 100644
--- a/drivers/staging/speakup/main.c
+++ b/drivers/staging/speakup/main.c

@@ -2091,7 +2091,8 @@ speakup_key(struct vc_data *vc, int shift_state, int keycode, u_short keysym,
 			spk_keydown = 0;
 			goto out;
 		}
-		value = spk_lastkey = pad_chars[value];
+		value = pad_chars[value];
+		spk_lastkey = value;
 		spk_keydown++;
 		spk_parked &= 0xfe;
 		goto no_map;

-- 
2.11.0

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help