* [PATCH] perf builtin-script: It's enough to include util/parse-options.h header once.
@ 2011-01-10 22:14 Jesper Juhl
2011-01-10 23:37 ` Arnaldo Carvalho de Melo
0 siblings, 1 reply; 2+ messages in thread
From: Jesper Juhl @ 2011-01-10 22:14 UTC (permalink / raw)
To: Peter Zijlstra
Cc: Paul Mackerras, Ingo Molnar, Arnaldo Carvalho de Melo,
linux-kernel, trivial
It should be enough to include util/parse-options.h once in
tools/perf/builtin-script.c
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
---
builtin-script.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
index 150a606..0173a80 100644
--- a/tools/perf/builtin-script.c
+++ b/tools/perf/builtin-script.c
@@ -8,11 +8,10 @@
#include "util/parse-options.h"
#include "util/session.h"
#include "util/symbol.h"
#include "util/thread.h"
#include "util/trace-event.h"
-#include "util/parse-options.h"
#include "util/util.h"
static char const *script_name;
static char const *generate_script_lang;
static bool debug_mode;
--
Jesper Juhl <jj@chaosbits.net> http://www.chaosbits.net/
Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] perf builtin-script: It's enough to include util/parse-options.h header once.
2011-01-10 22:14 [PATCH] perf builtin-script: It's enough to include util/parse-options.h header once Jesper Juhl
@ 2011-01-10 23:37 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 2+ messages in thread
From: Arnaldo Carvalho de Melo @ 2011-01-10 23:37 UTC (permalink / raw)
To: Jesper Juhl
Cc: Peter Zijlstra, Paul Mackerras, Ingo Molnar, linux-kernel, trivial
Em Mon, Jan 10, 2011 at 11:14:18PM +0100, Jesper Juhl escreveu:
> It should be enough to include util/parse-options.h once in
> tools/perf/builtin-script.c
Indeed, thanks!
> Signed-off-by: Jesper Juhl <jj@chaosbits.net>
> ---
> builtin-script.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
> index 150a606..0173a80 100644
> --- a/tools/perf/builtin-script.c
> +++ b/tools/perf/builtin-script.c
> @@ -8,11 +8,10 @@
> #include "util/parse-options.h"
> #include "util/session.h"
> #include "util/symbol.h"
> #include "util/thread.h"
> #include "util/trace-event.h"
> -#include "util/parse-options.h"
> #include "util/util.h"
>
> static char const *script_name;
> static char const *generate_script_lang;
> static bool debug_mode;
>
>
> --
> Jesper Juhl <jj@chaosbits.net> http://www.chaosbits.net/
> Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
> Plain text mails only, please.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-01-10 23:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-10 22:14 [PATCH] perf builtin-script: It's enough to include util/parse-options.h header once Jesper Juhl
2011-01-10 23:37 ` Arnaldo Carvalho de Melo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®