perf inject: Don't use globals where not needed to
authorArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 1 Oct 2012 18:20:58 +0000 (15:20 -0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 2 Oct 2012 21:36:41 +0000 (18:36 -0300)
Some variables were global but used in just one function, so move it to
where it belongs.

Leftover from patch at the beggining of this series.

Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Namhyung Kim <namhyung@gmail.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/n/tip-9cer20zhw64wbxyb0zias82i@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-inject.c

index 3c9ab55..4688bea 100644 (file)
@@ -242,11 +242,6 @@ static int __cmd_inject(struct perf_inject *inject)
        return ret;
 }
 
-static const char * const report_usage[] = {
-       "perf inject [<options>]",
-       NULL
-};
-
 int cmd_inject(int argc, const char **argv, const char *prefix __maybe_unused)
 {
        struct perf_inject inject = {
@@ -273,14 +268,18 @@ int cmd_inject(int argc, const char **argv, const char *prefix __maybe_unused)
                         "be more verbose (show build ids, etc)"),
                OPT_END()
        };
+       const char * const inject_usage[] = {
+               "perf inject [<options>]",
+               NULL
+       };
 
-       argc = parse_options(argc, argv, options, report_usage, 0);
+       argc = parse_options(argc, argv, options, inject_usage, 0);
 
        /*
         * Any (unrecognized) arguments left?
         */
        if (argc)
-               usage_with_options(report_usage, options);
+               usage_with_options(inject_usage, options);
 
        if (symbol__init() < 0)
                return -1;