Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1446085

[tip:perf/core] perf jit: Add missing curly braces

Show all headers | View raw

Commit-ID:  e03141db361399619f9ee97e00d4c6fe2b472104
Gitweb:     http://git.kernel.org/tip/e03141db361399619f9ee97e00d4c6fe2b472104
Author:     Dan Carpenter <dan.carpenter@oracle.com>
AuthorDate: Sat, 16 Jul 2016 00:07:12 +0300
Committer:  Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Mon, 18 Jul 2016 12:20:00 -0300

perf jit: Add missing curly braces

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>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: kernel-janitors@vger.kernel.org
Link: http://lkml.kernel.org/r/20160715210712.GA19522@mwanda
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/jvmti/jvmti_agent.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

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 — Previous 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