* [PATCH] perf: Remove unneeded include
@ 2013-11-06 22:20 Rodrigo Campos
2013-11-07 7:07 ` Namhyung Kim
2013-11-07 15:31 ` [tip:perf/core] perf tools: " tip-bot for Rodrigo Campos
0 siblings, 2 replies; 3+ messages in thread
From: Rodrigo Campos @ 2013-11-06 22:20 UTC (permalink / raw)
To: linux-kernel; +Cc: a.p.zijlstra, paulus, mingo, acme, Rodrigo Campos
There is no point in sort.h including itself.
The include was added when the file was created, in commit "perf tools: Create
util/sort.and use it" (dd68ada2d) and added a include to "sort.h" in lot of
files (all the files that started using the file). It was probably added by
mistake on sort.h too.
Signed-off-by: Rodrigo Campos <rodrigo@sdfg.com.ar>
---
tools/perf/util/sort.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/tools/perf/util/sort.h b/tools/perf/util/sort.h
index f4cc147..43e5ff4 100644
--- a/tools/perf/util/sort.h
+++ b/tools/perf/util/sort.h
@@ -22,7 +22,6 @@
#include "parse-events.h"
#include "thread.h"
-#include "sort.h"
extern regex_t parent_regex;
extern const char *sort_order;
--
1.8.4.rc3
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] perf: Remove unneeded include
2013-11-06 22:20 [PATCH] perf: Remove unneeded include Rodrigo Campos
@ 2013-11-07 7:07 ` Namhyung Kim
2013-11-07 15:31 ` [tip:perf/core] perf tools: " tip-bot for Rodrigo Campos
1 sibling, 0 replies; 3+ messages in thread
From: Namhyung Kim @ 2013-11-07 7:07 UTC (permalink / raw)
To: Rodrigo Campos; +Cc: linux-kernel, a.p.zijlstra, paulus, mingo, acme
Hi Rodrigo,
On Wed, 6 Nov 2013 22:20:54 +0000, Rodrigo Campos wrote:
> There is no point in sort.h including itself.
>
> The include was added when the file was created, in commit "perf tools: Create
> util/sort.and use it" (dd68ada2d) and added a include to "sort.h" in lot of
> files (all the files that started using the file). It was probably added by
> mistake on sort.h too.
Acked-by: Namhyung Kim <namhyung@kernel.org>
Thanks,
Namhyung
^ permalink raw reply [flat|nested] 3+ messages in thread
* [tip:perf/core] perf tools: Remove unneeded include
2013-11-06 22:20 [PATCH] perf: Remove unneeded include Rodrigo Campos
2013-11-07 7:07 ` Namhyung Kim
@ 2013-11-07 15:31 ` tip-bot for Rodrigo Campos
1 sibling, 0 replies; 3+ messages in thread
From: tip-bot for Rodrigo Campos @ 2013-11-07 15:31 UTC (permalink / raw)
To: linux-tip-commits
Cc: acme, linux-kernel, paulus, mingo, hpa, mingo, a.p.zijlstra,
rodrigo, namhyung, tglx
Commit-ID: 8ce000e83848578a621d64eccdc88bd34c2fc70c
Gitweb: http://git.kernel.org/tip/8ce000e83848578a621d64eccdc88bd34c2fc70c
Author: Rodrigo Campos <rodrigo@sdfg.com.ar>
AuthorDate: Wed, 6 Nov 2013 22:20:54 +0000
Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Thu, 7 Nov 2013 11:51:19 -0300
perf tools: Remove unneeded include
There is no point in sort.h including itself.
The include was added when the file was created, in commit "perf tools:
Create util/sort.and use it" (dd68ada2d) and added a include to "sort.h"
in lot of files (all the files that started using the file). It was
probably added by mistake on sort.h too.
Signed-off-by: Rodrigo Campos <rodrigo@sdfg.com.ar>
Acked-by: Namhyung Kim <namhyung@kernel.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1383776454-10595-1-git-send-email-rodrigo@sdfg.com.ar
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
tools/perf/util/sort.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/tools/perf/util/sort.h b/tools/perf/util/sort.h
index f4cc147..43e5ff4 100644
--- a/tools/perf/util/sort.h
+++ b/tools/perf/util/sort.h
@@ -22,7 +22,6 @@
#include "parse-events.h"
#include "thread.h"
-#include "sort.h"
extern regex_t parent_regex;
extern const char *sort_order;
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-11-07 15:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-06 22:20 [PATCH] perf: Remove unneeded include Rodrigo Campos
2013-11-07 7:07 ` Namhyung Kim
2013-11-07 15:31 ` [tip:perf/core] perf tools: " tip-bot for Rodrigo Campos
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).