Re: [Outreachy kernel] [PATCH 1/2] staging: speakup: Moved AND operator to previous line.

Path	csiph.com!weretis.net!feeder4.news.weretis.net!news.unit0.net!news.panservice.it!bofh.it!news.nic.it!robomod
From	Julia Lawall <julia.lawall@lip6.fr>
Newsgroups	linux.kernel
Subject	Re: [Outreachy kernel] [PATCH 1/2] staging: speakup: Moved AND operator to previous line.
Date	Sun, 19 Mar 2017 13:20:02 +0100
Message-ID	<tmHYu-5sy-9@gated-at.bofh.it> (permalink)
References	<tmDrQ-2j5-5@gated-at.bofh.it> <tmDrQ-2j5-3@gated-at.bofh.it>
X-Original-To	Arushi Singhal <arushisinghal19971997@gmail.com>
X-Ironport-Av	E=Sophos;i="5.36,188,1486422000"; d="scan'208";a="265179415"
X-X-Sender	jll@hadrien
User-Agent	Alpine 2.20 (DEB 67 2015-01-07)
MIME-Version	1.0
Content-Type	text/plain; charset=US-ASCII
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	49
Organization	linux.* mail to news gateway
X-Original-Cc	w.d.hubbs@gmail.com, chris@the-brannons.com, outreachy-kernel@googlegroups.com, kirk@reisers.ca, samuel.thibault@ens-lyon.org, gregkh@linuxfoundation.org, speakup@linux-speakup.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
X-Original-Date	Sun, 19 Mar 2017 13:13:59 +0100 (CET)
X-Original-Message-ID	<alpine.DEB.2.20.1703191313290.2108@hadrien>
X-Original-References	<20170319072403.10354-1-arushisinghal19971997@gmail.com> <20170319072403.10354-2-arushisinghal19971997@gmail.com>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1603968

Show key headers only | View raw


On Sun, 19 Mar 2017, Arushi Singhal wrote:

> Moved logical AND operator to previous line to fix the following
> checkpatch issue:

You seem to have AND and OR confused.  AND is &&, OR is ||.  This patch is
about OR, not AND.

julia

>
> CHECK: Logical continuations should be on the previous line.
>
> Signed-off-by: Arushi Singhal <arushisinghal19971997@gmail.com>
> ---
>  drivers/staging/speakup/main.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/speakup/main.c b/drivers/staging/speakup/main.c
> index c10445624e92..def1a36da9dd 100644
> --- a/drivers/staging/speakup/main.c
> +++ b/drivers/staging/speakup/main.c
> @@ -2173,10 +2173,10 @@ speakup_key(struct vc_data *vc, int shift_state, int keycode, u_short keysym,
>  	if (up_flag || spk_killed || type == KT_SHIFT)
>  		goto out;
>  	spk_shut_up &= 0xfe;
> -	kh = (value == KVAL(K_DOWN))
> -	    || (value == KVAL(K_UP))
> -	    || (value == KVAL(K_LEFT))
> -	    || (value == KVAL(K_RIGHT));
> +	kh = (value == KVAL(K_DOWN)) ||
> +	    (value == KVAL(K_UP)) ||
> +	    (value == KVAL(K_LEFT)) ||
> +	    (value == KVAL(K_RIGHT));
>  	if ((cursor_track != read_all_mode) || !kh)
>  		if (!spk_no_intr)
>  			spk_do_flush();
> --
> 2.11.0
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20170319072403.10354-2-arushisinghal19971997%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>

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

Thread

[PATCH 1/2] staging: speakup: Moved AND operator to previous line. Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-03-19 08:30 +0100
  Re: [Outreachy kernel] [PATCH 1/2] staging: speakup: Moved AND  operator to previous line. Julia Lawall <julia.lawall@lip6.fr> - 2017-03-19 13:20 +0100

csiph-web