Date: Tue, 26 May 2015 12:40:33 +0200 (CEST)
From: Thomas Gleixner <tglx@linutronix.de>
To: Andrea Parri <parri.andrea@gmail.com>
cc: john.stultz@linaro.org, linux-kernel@vger.kernel.org, trivial@kernel.org
Subject: Re: [PATCH trivial] time/timer.c: coding style fix
In-Reply-To: <20150526101719.GA5724@andrea>
Message-ID: <alpine.DEB.2.11.1505261238590.5457@nanos>
References: <20150526101719.GA5724@andrea>
User-Agent: Alpine 2.11 (DEB 23 2013-08-11)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1152
Lines: 37

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@gmail.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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/