linux-kernel - Re: [PATCH trivial] time/timer.c: coding style fix

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 linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 26 May 2015 12:40:33 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Andrea Parri <parri.andrea@...il.com>
cc:	john.stultz@...aro.org, linux-kernel@...r.kernel.org,
	trivial@...nel.org
Subject: Re: [PATCH trivial] time/timer.c: coding style fix

On Tue, 26 May 2015, Andrea Parri wrote:

> Put the opening brace last on the line in switch statement.

This is completeley pointless churn. We do such changes when we change
the code not just to make some random codign style checker happy.

Thanks,

	tglx

 
> Signed-off-by: Andrea Parri <parri.andrea@...il.com>
> ---
>  kernel/time/timer.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/kernel/time/timer.c b/kernel/time/timer.c
> index 2ece3aa..19e61f2 100644
> --- a/kernel/time/timer.c
> +++ b/kernel/time/timer.c
> @@ -1473,8 +1473,7 @@ signed long __sched schedule_timeout(signed long timeout)
>         struct timer_list timer;
>         unsigned long expire;
>  
> -       switch (timeout)
> -       {
> +       switch (timeout) {
>         case MAX_SCHEDULE_TIMEOUT:
>                 /*
>                  * These two special cases are useful to be comfortable
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/