perf stat report: Move csv_sep initialization before report command
authorJiri Olsa <jolsa@kernel.org>
Thu, 5 Nov 2015 14:41:01 +0000 (15:41 +0100)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Thu, 17 Dec 2015 19:29:06 +0000 (16:29 -0300)
So we have csv_sep properly initialized before report command leg.

Reported-by: Kan Liang <kan.liang@intel.com>
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1446734469-11352-18-git-send-email-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-stat.c

index c780525800236cc7fd465ebfe93225e4b8b39a7c..f9d4e0963ac75a84d835a2eb283968bc27a33b2f 100644 (file)
@@ -1776,6 +1776,13 @@ int cmd_stat(int argc, const char **argv, const char *prefix __maybe_unused)
                                        (const char **) stat_usage,
                                        PARSE_OPT_STOP_AT_NON_OPTION);
 
+       if (csv_sep) {
+               csv_output = true;
+               if (!strcmp(csv_sep, "\\t"))
+                       csv_sep = "\t";
+       } else
+               csv_sep = DEFAULT_SEPARATOR;
+
        if (argc && !strncmp(argv[0], "rec", 3)) {
                argc = __cmd_record(argc, argv);
                if (argc < 0)
@@ -1826,13 +1833,6 @@ int cmd_stat(int argc, const char **argv, const char *prefix __maybe_unused)
 
        stat_config.output = output;
 
-       if (csv_sep) {
-               csv_output = true;
-               if (!strcmp(csv_sep, "\\t"))
-                       csv_sep = "\t";
-       } else
-               csv_sep = DEFAULT_SEPARATOR;
-
        /*
         * let the spreadsheet do the pretty-printing
         */