linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] perf evlist: Remove trailing semicolon
@ 2018-01-11 15:50 Luis de Bethencourt
  2018-01-17 16:25 ` [tip:perf/core] " tip-bot for Luis de Bethencourt
  0 siblings, 1 reply; 2+ messages in thread
From: Luis de Bethencourt @ 2018-01-11 15:50 UTC (permalink / raw)
  To: linux-kernel
  Cc: Peter Zijlstra, Ingo Molnar, Joe Perches,
	Arnaldo Carvalho de Melo, Alexander Shishkin, Jiri Olsa,
	Namhyung Kim, Luis de Bethencourt

The trailing semicolon is an empty statement that does no operation.
Removing it since it doesn't do anything.

Signed-off-by: Luis de Bethencourt <luisbg@kernel.org>
---

Hi,

After fixing the same thing in drivers/staging/rtl8723bs/, Joe Perches
suggested I fix it treewide [0].

Best regards 
Luis


[0] http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2018-January/115410.html
[1] http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2018-January/115390.html

 tools/perf/util/evlist.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c
index f0a5e09c4071..120efd85f2c8 100644
--- a/tools/perf/util/evlist.c
+++ b/tools/perf/util/evlist.c
@@ -1760,7 +1760,7 @@ void perf_evlist__toggle_bkw_mmap(struct perf_evlist *evlist,
 	switch (old_state) {
 	case BKW_MMAP_NOTREADY: {
 		if (state != BKW_MMAP_RUNNING)
-			goto state_err;;
+			goto state_err;
 		break;
 	}
 	case BKW_MMAP_RUNNING: {
-- 
2.15.1

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

* [tip:perf/core] perf evlist: Remove trailing semicolon
  2018-01-11 15:50 [PATCH] perf evlist: Remove trailing semicolon Luis de Bethencourt
@ 2018-01-17 16:25 ` tip-bot for Luis de Bethencourt
  0 siblings, 0 replies; 2+ messages in thread
From: tip-bot for Luis de Bethencourt @ 2018-01-17 16:25 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: peterz, hpa, namhyung, luisbg, acme, joe, alexander.shishkin,
	linux-kernel, jolsa, tglx, mingo

Commit-ID:  dd8bd53ab86133327412e74bf5ba31a8ec2826d4
Gitweb:     https://git.kernel.org/tip/dd8bd53ab86133327412e74bf5ba31a8ec2826d4
Author:     Luis de Bethencourt <luisbg@kernel.org>
AuthorDate: Thu, 11 Jan 2018 15:50:20 +0000
Committer:  Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Fri, 12 Jan 2018 11:02:55 -0300

perf evlist: Remove trailing semicolon

The trailing semicolon is an empty statement that does no operation.
Removing it since it doesn't do anything.

Signed-off-by: Luis de Bethencourt <luisbg@kernel.org>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Joe Perches <joe@perches.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Link: http://lkml.kernel.org/r/20180111155020.9782-1-luisbg@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/util/evlist.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c
index f0a5e09..120efd8 100644
--- a/tools/perf/util/evlist.c
+++ b/tools/perf/util/evlist.c
@@ -1760,7 +1760,7 @@ void perf_evlist__toggle_bkw_mmap(struct perf_evlist *evlist,
 	switch (old_state) {
 	case BKW_MMAP_NOTREADY: {
 		if (state != BKW_MMAP_RUNNING)
-			goto state_err;;
+			goto state_err;
 		break;
 	}
 	case BKW_MMAP_RUNNING: {

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

end of thread, other threads:[~2018-01-17 16:28 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-11 15:50 [PATCH] perf evlist: Remove trailing semicolon Luis de Bethencourt
2018-01-17 16:25 ` [tip:perf/core] " tip-bot for Luis de Bethencourt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).