From: Jeffrin Jose <ahiliat...@yahoo.co.in>
Date: Tue,  7 Aug 2012 19:30:20 +0530

> Fixed coding style issue relating to indentation in
> net/appletalk/ddp.c found by checkpatch.pl tool
> 
> Signed-off-by: Jeffrin Jose <ahiliat...@yahoo.co.in>

Please just stop trying to fix this code up, it's noise and you
keep making mistakes.

>       case TIOCOUTQ: {
> -                     long amount = sk->sk_sndbuf - sk_wmem_alloc_get(sk);
> +             long amount = sk->sk_sndbuf - sk_wmem_alloc_get(sk);
>  
> -                     if (amount < 0)
> -                             amount = 0;
> -                     rc = put_user(amount, (int __user *)argp);
> -                     break;
> +             if (amount < 0)
> +                     amount = 0;
> +             rc = put_user(amount, (int __user *)argp);
> +             break;
>               }

This time you didn't adjust the closing brace, as needed.

I'm not applying this and I seriously discourage you from pursing this
further, it's better you spend your time learning how the kernel works
rather than suffering through this coding style cleanup business.

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to