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

Groups > linux.kernel > #1733090

Re: [Outreachy kernel] [PATCH v2] Staging: irda: Don't use assignment inside if statement

Show all headers | View raw

On Sat, 16 Sep 2017, Srishti Sharma wrote:

> On Sat, Sep 16, 2017 at 1:30 AM, Julia Lawall <julia.lawall@lip6.fr> wrote:
> >
> >
> > On Sat, 16 Sep 2017, Srishti Sharma wrote:
> >
> >> Write assignment statement outside 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>
> >
> > Acked-by: Julia Lawall <julia.lawall@lip6.fr>
>
> Have sent a patchset for this instead .

Thanks.  Actually, you should have copied the ack.

julia

>
> Regards,
> Srishti
> >
> >> ---
> >> Changes in v2:
> >>  -Semicolon was missing in one of the statements of the
> >>   semantic patch
> >>
> >>  drivers/staging/irda/drivers/irda-usb.c | 4 ++--
> >>  drivers/staging/irda/drivers/mcs7780.c  | 9 ++++++---
> >>  drivers/staging/irda/net/irqueue.c      | 3 ++-
> >>  3 files changed, 10 insertions(+), 6 deletions(-)
> >>
> >> diff --git a/drivers/staging/irda/drivers/irda-usb.c b/drivers/staging/irda/drivers/irda-usb.c
> >> index 723e49b..82bfc05 100644
> >> --- a/drivers/staging/irda/drivers/irda-usb.c
> >> +++ b/drivers/staging/irda/drivers/irda-usb.c
> >> @@ -334,9 +334,9 @@ static void irda_usb_change_speed_xbofs(struct irda_usb_cb *self)
> >>       urb->transfer_flags = 0;
> >>
> >>       /* Irq disabled -> GFP_ATOMIC */
> >> -     if ((ret = usb_submit_urb(urb, GFP_ATOMIC))) {
> >> +     ret = usb_submit_urb(urb, GFP_ATOMIC);
> >> +     if (ret)
> >>               net_warn_ratelimited("%s(), failed Speed URB\n", __func__);
> >> -     }
> >>  }
> >>
> >>  /*------------------------------------------------------------------*/
> >> diff --git a/drivers/staging/irda/drivers/mcs7780.c b/drivers/staging/irda/drivers/mcs7780.c
> >> index c3f0b25..2b674d5 100644
> >> --- a/drivers/staging/irda/drivers/mcs7780.c
> >> +++ b/drivers/staging/irda/drivers/mcs7780.c
> >> @@ -605,19 +605,22 @@ static int mcs_speed_change(struct mcs_cb *mcs)
> >>       if (mcs->new_speed <= 115200) {
> >>               rval &= ~MCS_FIR;
> >>
> >> -             if ((rst = (mcs->speed > 115200)))
> >> +             rst = (mcs->speed > 115200);
> >> +             if (rst)
> >>                       mcs_set_reg(mcs, MCS_MINRXPW_REG, 0);
> >>
> >>       } else if (mcs->new_speed <= 1152000) {
> >>               rval &= ~MCS_FIR;
> >>
> >> -             if ((rst = !(mcs->speed == 576000 || mcs->speed == 1152000)))
> >> +             rst = !(mcs->speed == 576000 || mcs->speed == 1152000);
> >> +             if (rst)
> >>                       mcs_set_reg(mcs, MCS_MINRXPW_REG, 5);
> >>
> >>       } else {
> >>               rval |= MCS_FIR;
> >>
> >> -             if ((rst = (mcs->speed != 4000000)))
> >> +             rst = (mcs->speed != 4000000);
> >> +             if (rst)
> >>                       mcs_set_reg(mcs, MCS_MINRXPW_REG, 5);
> >>
> >>       }
> >> diff --git a/drivers/staging/irda/net/irqueue.c b/drivers/staging/irda/net/irqueue.c
> >> index 160dc89..5aab072 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;
> >>       }
> >> --
> >> 2.7.4
> >>
> >> --
> >> 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/1505504680-22167-1-git-send-email-srishtishar%40gmail.com.
> >> For more options, visit https://groups.google.com/d/optout.
> >>
>
> --
> 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/CAB3L5oxr5K5aMo06ROCVV9ejY6a_g1%2BuUkjU%3DZ5bKTJtzUs7sA%40mail.gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>

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

Thread

[PATCH v2] Staging: irda: Don't use assignment inside if statement Srishti Sharma <srishtishar@gmail.com> - 2017-09-15 21:50 +0200
  Re: [Outreachy kernel] [PATCH v2] Staging: irda: Don't use assignment  inside if statement Julia Lawall <julia.lawall@lip6.fr> - 2017-09-15 22:10 +0200
    Re: [Outreachy kernel] [PATCH v2] Staging: irda: Don't use assignment  inside if statement Srishti Sharma <srishtishar@gmail.com> - 2017-09-15 23:10 +0200
      Re: [Outreachy kernel] [PATCH v2] Staging: irda: Don't use assignment  inside if statement Julia Lawall <julia.lawall@lip6.fr> - 2017-09-15 23:10 +0200
        Re: [Outreachy kernel] [PATCH v2] Staging: irda: Don't use assignment  inside if statement Srishti Sharma <srishtishar@gmail.com> - 2017-09-15 23:20 +0200

csiph-web