[patch 1/2] perf jit: add missing curly braces

From	Dan Carpenter <dan.carpenter@oracle.com>
Newsgroups	linux.kernel
Subject	[patch 1/2] perf jit: add missing curly braces
Date	2016-07-15 23:10 +0200
Message-ID	<rViwV-3rh-19@gated-at.bofh.it> (permalink)
Organization	linux.* mail to news gateway

Show all headers | View raw

It doesn't change the runtime behavior, but my static checker complains
that curly braces were intended.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>

diff --git a/tools/perf/jvmti/jvmti_agent.c b/tools/perf/jvmti/jvmti_agent.c
index 3573f31..91bf333 100644
--- a/tools/perf/jvmti/jvmti_agent.c
+++ b/tools/perf/jvmti/jvmti_agent.c
@@ -491,10 +491,11 @@ jvmti_write_debug_info(void *agent, uint64_t code, const char *file,
 		if (sret != 1)
 			goto error;
 	}
-	if (padding_count)
+	if (padding_count) {
 		sret = fwrite_unlocked(pad_bytes, padding_count, 1, fp);
 		if (sret != 1)
 			goto error;
+	}
 
 	funlockfile(fp);
 	return 0;

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[patch 1/2] perf jit: add missing curly braces Dan Carpenter <dan.carpenter@oracle.com> - 2016-07-15 23:10 +0200
  Re: [patch 1/2] perf jit: add missing curly braces Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-07-16 00:30 +0200
  Re: [patch 1/2] perf jit: add missing curly braces Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-07-16 00:30 +0200
  Re: [patch 1/2] perf jit: add missing curly braces Stephane Eranian <eranian@google.com> - 2016-07-16 18:40 +0200
  [tip:perf/core] perf jit: Add missing curly braces tip-bot for Dan Carpenter <tipbot@zytor.com> - 2016-07-19 09:00 +0200

csiph-web