lkml.org 
[lkml]   [2016]   [Jul]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 6/9] staging: ks7010: Delete unnecessary braces
    On Sun, Jul 17, 2016 at 08:55:41PM +0200, Julia Lawall wrote:
    >
    >
    > On Sun, 17 Jul 2016, SF Markus Elfring wrote:
    >
    > > From: Markus Elfring <elfring@users.sourceforge.net>
    > > Date: Sun, 17 Jul 2016 18:39:03 +0200
    > >
    > > Do not use curly brackets at some source code places
    > > where a single statement should be sufficient.
    > >
    > > Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
    > > ---
    > > drivers/staging/ks7010/ks7010_sdio.c | 48 ++++++++++++------------------------
    > > 1 file changed, 16 insertions(+), 32 deletions(-)
    > >
    > > diff --git a/drivers/staging/ks7010/ks7010_sdio.c b/drivers/staging/ks7010/ks7010_sdio.c
    > > index b3ca8e2..1e072e3 100644
    > > --- a/drivers/staging/ks7010/ks7010_sdio.c
    > > +++ b/drivers/staging/ks7010/ks7010_sdio.c
    > > @@ -109,9 +109,8 @@ void ks_wlan_hw_sleep_doze_request(struct ks_wlan_private *priv)
    > > DPRINTK(3, "sleep_mode=SLP_SLEEP\n");
    > > atomic_set(&priv->sleepstatus.status, 1);
    > > priv->last_doze = jiffies;
    > > - } else {
    > > + } else
    > > DPRINTK(1, "sleep_mode=%d\n", priv->sleep_mode);
    > > - }
    >
    > This is a violation of CodingStyle:

    Julia is right again, I agree.

    [unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2016-07-20 18:41    [W:4.465 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site