perf evlist: Simplify set_maps() logic
authorAdrian Hunter <adrian.hunter@intel.com>
Tue, 8 Sep 2015 07:58:50 +0000 (10:58 +0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
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 <adrian.hunter@intel.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Kan Liang <kan.liang@intel.com>
Link: http://lkml.kernel.org/r/1441699142-18905-3-git-send-email-adrian.hunter@intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
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);