public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH trivial] time/timer.c: coding style fix
@ 2015-05-26 10:17 Andrea Parri
  2015-05-26 10:40 ` Thomas Gleixner
  0 siblings, 1 reply; 2+ messages in thread
From: Andrea Parri @ 2015-05-26 10:17 UTC (permalink / raw)
  To: john.stultz; +Cc: tglx, linux-kernel, trivial


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

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

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH trivial] time/timer.c: coding style fix
  2015-05-26 10:17 [PATCH trivial] time/timer.c: coding style fix Andrea Parri
@ 2015-05-26 10:40 ` Thomas Gleixner
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Gleixner @ 2015-05-26 10:40 UTC (permalink / raw)
  To: Andrea Parri; +Cc: john.stultz, linux-kernel, trivial

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
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-05-26 13:00 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-26 10:17 [PATCH trivial] time/timer.c: coding style fix Andrea Parri
2015-05-26 10:40 ` Thomas Gleixner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox