Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1605539

[PATCH 05/11] staging: speakup: Remove multiple assignments

From Arushi Singhal <arushisinghal19971997@gmail.com>
Newsgroups linux.kernel
Subject [PATCH 05/11] staging: speakup: Remove multiple assignments
Date 2017-03-21 12:50 +0100
Message-ID <tnqsz-2K1-47@gated-at.bofh.it> (permalink)
References <tnqsy-2K1-7@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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

Signed-off-by: Arushi Singhal <arushisinghal19971997@gmail.com>
---
 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 21e76b031449..c10445624e92 100644
--- a/drivers/staging/speakup/main.c
+++ b/drivers/staging/speakup/main.c
@@ -2106,7 +2106,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

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[RESEND PATCH 00/11] staging:speakup: Multiple Checkpatch issues. Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
  [PATCH 09/11] staging: speakup: Simplify the NULL comparisons Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
    Re: [PATCH 09/11] staging: speakup: Simplify the NULL comparisons Greg KH <gregkh@linuxfoundation.org> - 2017-03-23 14:20 +0100
  [PATCH 11/11] staging: speakup: Fix alignment with parenthesis. Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
    Re: [PATCH 11/11] staging: speakup: Fix alignment with parenthesis. Greg KH <gregkh@linuxfoundation.org> - 2017-03-23 14:30 +0100
  [PATCH 06/11] staging: speakup: Moved OR operator to previous line. Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
  [PATCH 07/11] staging: speakup: spaces preferred around operator Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
  [PATCH 03/11] staging: speakup: Simplify "NULL" comparisons Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
  [PATCH 08/11] staging: speakup: Removed Unnecessary parentheses. Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
  [PATCH 04/11] staging: speakup: fixes braces {} should be used on all arms of this statement Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
  [PATCH 10/11] staging: speakup: Match alignment with open parenthesis. Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
  [PATCH 02/11] staging: speakup: Remove multiple assignments Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
    Re: [PATCH 02/11] staging: speakup: Remove multiple assignments Greg KH <gregkh@linuxfoundation.org> - 2017-03-23 14:20 +0100
  [PATCH 01/11] staging: speakup: Moved logical to previous line. Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
  [PATCH 05/11] staging: speakup: Remove multiple assignments Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-21 12:50 +0100
    Re: [PATCH 05/11] staging: speakup: Remove multiple assignments Greg KH <gregkh@linuxfoundation.org> - 2017-03-23 14:20 +0100
  Re: [RESEND PATCH 00/11] staging:speakup: Multiple Checkpatch issues. Greg KH <gregkh@linuxfoundation.org> - 2017-03-22 21:50 +0100

csiph-web