On Wed, 2015-07-29 at 21:34 -0400, Dave Perez wrote: > This is a patch to the rtw_debug.c file that fixes styling errors relating to new lines after variable declarations. [] > diff --git a/drivers/staging/rtl8188eu/core/rtw_debug.c b/drivers/staging/rtl8188eu/core/rtw_debug.c [] > @@ -219,6 +219,7 @@ int proc_get_ht_option(char *page, char **start, > struct mlme_priv *pmlmepriv = &(padapter->mlmepriv); > > int len = 0; > + > len += snprintf(page + len, count - len, "ht_option=%d\n", pmlmepriv->htpriv.ht_option); > *eof = 1; > return len; the blank line before "int len = 0" could also be removed. The routine _could_ be rewritten more simply as: struct mlme_priv *pmlmepriv = &padapter->mlmepriv; int len = snprintf(page, count, "ht_option=%d\n", pmlmepriv->htpriv.htoption); *eof = 1; return len; } but there are a bunch of other routines that use this form so it's probably fine as-is. _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
