perf tools: Get rid of superfluos call to reset_dimensions
authorJiri Olsa <jolsa@kernel.org>
Tue, 6 Oct 2015 12:25:10 +0000 (14:25 +0200)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 6 Oct 2015 21:04:58 +0000 (18:04 -0300)
There's no need to call reset_dimensions within __setup_output_field
function. It's already called in its caller setup_sorting right before
perf_hpp__init, which will be changed in following patch to respect
taken dimension.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Acked-by: Namhyung Kim <namhyung@kernel.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1444134312-29136-2-git-send-email-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/sort.c

index ee94b728fca4b3a4e10df847ca7ee26e663fa844..8521e3c89b8d5ef6f0a553e87c9821d33da15789 100644 (file)
@@ -1917,8 +1917,6 @@ static int __setup_output_field(void)
        if (field_order == NULL)
                return 0;
 
-       reset_dimensions();
-
        strp = str = strdup(field_order);
        if (str == NULL) {
                error("Not enough memory to setup output fields");