On Fri, Sep 22, 2017 at 11:52:36AM +0200, Julia Lawall wrote:
> 
> 
> On Fri, 22 Sep 2017, Aishwarya Pant wrote:
> 
> > Implicit type conversions are bad; they hinder readability of code and have
> > potential to cause bugs. Here the variable wait_ack is always supplied a 
> > bool
> > value while in function declarations it is defined as an int type. Fix it by
> > defining wait_ack a bool type in all usages.
> 
> These are two patches on the same file.  They need to be in a series, so
> Greg doesn't have to figure out whether they depend on each other.
> 
> Also, since you are changing the function headers, it would be very nice
> if you could also change them to respect the 80 character limit.

Okay I'll create a patch series on rtl8188eu cleanups. Please drop this patch.

> 
> julia
> 
> >
> > Signed-off-by: Aishwarya Pant <aishp...@gmail.com>
> > ---
> >  drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c 
> > b/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c
> > index 2db2ca6..99f3d5c 100644
> > --- a/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c
> > +++ b/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c
> > @@ -1196,7 +1196,7 @@ static void issue_assocreq(struct adapter *padapter)
> >  }
> >
> >  /* when wait_ack is true, this function should be called at process 
> > context */
> > -static int _issue_nulldata(struct adapter *padapter, unsigned char *da, 
> > unsigned int power_mode, int wait_ack)
> > +static int _issue_nulldata(struct adapter *padapter, unsigned char *da, 
> > unsigned int power_mode, bool wait_ack)
> >  {
> >     int ret = _FAIL;
> >     struct xmit_frame                       *pmgntframe;
> > @@ -1316,7 +1316,7 @@ int issue_nulldata(struct adapter *padapter, unsigned 
> > char *da, unsigned int pow
> >  }
> >
> >  /* when wait_ack is true, this function should be called at process 
> > context */
> > -static int _issue_qos_nulldata(struct adapter *padapter, unsigned char 
> > *da, u16 tid, int wait_ack)
> > +static int _issue_qos_nulldata(struct adapter *padapter, unsigned char 
> > *da, u16 tid, bool wait_ack)
> >  {
> >     int ret = _FAIL;
> >     struct xmit_frame                       *pmgntframe;
> > @@ -1442,7 +1442,7 @@ int issue_qos_nulldata(struct adapter *padapter, 
> > unsigned char *da, u16 tid, int
> >     return ret;
> >  }
> >
> > -static int _issue_deauth(struct adapter *padapter, unsigned char *da, 
> > unsigned short reason, u8 wait_ack)
> > +static int _issue_deauth(struct adapter *padapter, unsigned char *da, 
> > unsigned short reason, bool wait_ack)
> >  {
> >     struct xmit_frame                       *pmgntframe;
> >     struct pkt_attrib                       *pattrib;
> > --
> > 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+unsubscr...@googlegroups.com.
> > To post to this group, send email to outreachy-ker...@googlegroups.com.
> > To view this discussion on the web visit 
> > https://groups.google.com/d/msgid/outreachy-kernel/20170922093939.GA29152%40aishwarya.
> > For more options, visit https://groups.google.com/d/optout.
> >
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to