lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Sun, 08 Mar 2020 13:19:27 -0700
From:   Joe Perches <joe@...ches.com>
To:     Shreeya Patel <shreeya.patel23498@...il.com>,
        Julia Lawall <julia.lawall@...ia.fr>
Cc:     Larry.Finger@...inger.net, gregkh@...uxfoundation.org,
        devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org,
        outreachy-kernel@...glegroups.com, sbrivio@...hat.com,
        daniel.baluta@...il.com, nramas@...ux.microsoft.com,
        hverkuil@...all.nl
Subject: Re: [Outreachy kernel] [PATCH] Staging: rtl8188eu: Add space around
 operator

On Mon, 2020-03-09 at 01:40 +0530, Shreeya Patel wrote:
> On Sun, 2020-03-08 at 12:59 -0700, Joe Perches wrote:
> Hi Joe,

Hello.
[]
> > > > diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme.c
> > > > b/drivers/staging/rtl8188eu/core/rtw_mlme.c
> > []
> > > > @@ -924,7 +924,7 @@ static void rtw_joinbss_update_network(struct
> > > > adapter *padapter, struct wlan_net
> > > >  	/* update fw_state will clr _FW_UNDER_LINKING here indirectly
> > > > */
> > > >  	switch (pnetwork->network.InfrastructureMode) {
> > > >  	case Ndis802_11Infrastructure:
> > > > -		if (pmlmepriv->fw_state&WIFI_UNDER_WPS)
> > > > +		if (pmlmepriv->fw_state & WIFI_UNDER_WPS)
> > > >  			pmlmepriv->fw_state =
> > > > WIFI_STATION_STATE|WIFI_UNDER_WPS;
> > 
> > Like adding spaces around the | here too.
> > 
> I thought of doing this but then it was introducing another warning of
> "Line over 80 charachters" that is why I didn't proceed with it.
[]
> What is your suggestion over it? Should I let the line be over 80
> characters and add spaces around the operators?

Just ignore the long line warnings, there are many already
existing long
lines in that subsystem.

That should be a style challenge for a later time.

$ git ls-files drivers/staging/rtl8188eu | \
  xargs awk '{ print length($0); }' | \
  sort | uniq -c | sort -rn -k2
      1 187
      1 180
      1 171
      1 166
      1 163
      1 159
      1 158
      2 157
      1 153
      2 151
      1 146
      1 145
      1 144
      1 143
      1 142
      2 141
      1 140
      1 139
      1 137
      1 135
      5 134
      6 132
      3 131
      3 130
      6 129
      3 128
      6 127
      6 126
      3 125
      3 124
      5 123
      4 122
      8 121
      6 120
      4 119
      7 118
     10 117
     11 116
      9 115
      5 114
     11 113
     13 112
      8 111
     17 110
     25 109
     24 108
     14 107
     20 106
     19 105
     34 104
     19 103
     26 102
     22 101
     22 100
     25 99
     20 98
     23 97
     33 96
     32 95
     43 94
     40 93
     49 92
     47 91
     51 90
     48 89
     55 88
     50 87
     37 86
     48 85
     57 84
     45 83
     61 82
     61 81


Powered by blists - more mailing lists