2015-05-26 13:00:24

by Andrea Parri

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

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

Signed-off-by: Andrea Parri <[email protected]>
---
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

2015-05-26 10:40:50

by Thomas Gleixner

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 <[email protected]>
> ---
> 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
>