perf evlist: Simplify set_maps() logic
authorAdrian Hunter <[email protected]>
Tue, 8 Sep 2015 07:58:50 +0000 (10:58 +0300)
committerArnaldo Carvalho de Melo <[email protected]>
Tue, 15 Sep 2015 13:15:39 +0000 (10:15 -0300)
Don't need to check for NULL when "putting" evlist->maps and
evlist->threads because the "put" functions already do that.

Signed-off-by: Adrian Hunter <[email protected]>
Acked-by: Jiri Olsa <[email protected]>
Cc: Kan Liang <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
tools/perf/util/evlist.c

index 95e07ea3904cf9b0de8179d979d07744a06b89c9..9cb9296cc4f8377c5a3eba02c32623749bc5a45a 100644 (file)
@@ -1156,14 +1156,10 @@ int perf_evlist__set_maps(struct perf_evlist *evlist,
                          struct cpu_map *cpus,
                          struct thread_map *threads)
 {
-       if (evlist->cpus)
-               cpu_map__put(evlist->cpus);
-
+       cpu_map__put(evlist->cpus);
        evlist->cpus = cpus;
 
-       if (evlist->threads)
-               thread_map__put(evlist->threads);
-
+       thread_map__put(evlist->threads);
        evlist->threads = threads;
 
        return perf_evlist__propagate_maps(evlist, false);