public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] proc: Trivial brace balancing
@ 2016-11-04 12:55 Christopher Covington
  2016-11-04 13:37 ` Alexey Dobriyan
  0 siblings, 1 reply; 2+ messages in thread
From: Christopher Covington @ 2016-11-04 12:55 UTC (permalink / raw)
  To: trivial, Andrew Morton, Alexey Dobriyan, Kees Cook,
	Thomas Gleixner, Andy Shevchenko, Joe Perches, Andy Lutomirski,
	Richard W.M. Jones, Christopher Covington, linux-kernel

As suggested by Documentation/CodingStyle right before section 3.1: Spaces,
use braces on all branches of a conditional statement if they are used on
any branch. No functional changes.

Signed-off-by: Christopher Covington <cov@codeaurora.org>
---
 fs/proc/array.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/fs/proc/array.c b/fs/proc/array.c
index 81818ad..9a3ca9e 100644
--- a/fs/proc/array.c
+++ b/fs/proc/array.c
@@ -552,8 +552,9 @@ static int do_task_stat(struct seq_file *m, struct pid_namespace *ns,
 		seq_put_decimal_ull(m, " ", mm->arg_end);
 		seq_put_decimal_ull(m, " ", mm->env_start);
 		seq_put_decimal_ull(m, " ", mm->env_end);
-	} else
+	} else {
 		seq_puts(m, " 0 0 0 0 0 0 0");
+	}
 
 	if (permitted)
 		seq_put_decimal_ll(m, " ", task->exit_code);
-- 
Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm
Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora
Forum, a Linux Foundation Collaborative Project.

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

* Re: [PATCH] proc: Trivial brace balancing
  2016-11-04 12:55 [PATCH] proc: Trivial brace balancing Christopher Covington
@ 2016-11-04 13:37 ` Alexey Dobriyan
  0 siblings, 0 replies; 2+ messages in thread
From: Alexey Dobriyan @ 2016-11-04 13:37 UTC (permalink / raw)
  To: Christopher Covington
  Cc: trivial, Andrew Morton, Kees Cook, Thomas Gleixner,
	Andy Shevchenko, Joe Perches, Andy Lutomirski, Richard W.M. Jones,
	Linux Kernel

On Fri, Nov 4, 2016 at 3:55 PM, Christopher Covington
<cov@codeaurora.org> wrote:

> -       } else
> +       } else {
>                 seq_puts(m, " 0 0 0 0 0 0 0");
> +       }

Oh, come on, people.

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

end of thread, other threads:[~2016-11-04 13:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-04 12:55 [PATCH] proc: Trivial brace balancing Christopher Covington
2016-11-04 13:37 ` Alexey Dobriyan

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