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

From: Thomas Gleixner
Date: Tue May 26 2015 - 09:09:47 EST


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