[PATCH 9/9] staging: csr: Move else keyword up to follow after }

Dan Carpenter dan.carpenter at oracle.com
Sat Jun 1 08:34:15 UTC 2013


On Fri, May 31, 2013 at 10:23:52PM -0700, Lisa Nguyen wrote:
> Resolved a closing brace issue generated by checkpatch.pl to
> meet kernel coding standards in sme_userspace.c
> 
> Signed-off-by: Lisa Nguyen <lisa at xenapiadmin.com>
> ---
>  drivers/staging/csr/sme_userspace.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/csr/sme_userspace.c b/drivers/staging/csr/sme_userspace.c
> index 7e9db10..7076d2b 100644
> --- a/drivers/staging/csr/sme_userspace.c
> +++ b/drivers/staging/csr/sme_userspace.c
> @@ -230,8 +230,7 @@ unifi_ta_indicate_protocol(void *ospriv,
>  			    packet_type,
>  			    direction,
>  			    *src_addr);
> -    }
> -    else {
> +    } else {

Oh...  It wouldn't have violated the "one thing per patch" to do
that in the previous patch.

regards,
dan carpenter




More information about the devel mailing list