perf evsel: Remove duplicate member zeroing after free

Stephane Eranian · acmel · commit de256a4e6b90 · 2014-01-20T16:19:09.000-03:00
No need to set evsel->fd to NULL after calling perf_evsel__free_fd(), as this method already does that. Cc: Adrian Hunter <adrian.hunter@intel.com> 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@kernel.org> Cc: Paul Mackerras <paulus@samba.org> Cc: Peter Zijlstra <peterz@infradead.org> Link: http://lkml.kernel.org/n/tip-wu6kul8fpapr8iyqm685ewtf@git.kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
diff --git a/tools/perf/util/evsel.c b/tools/perf/util/evsel.c
@@ -1081,7 +1081,6 @@ void perf_evsel__close(struct perf_evsel *evsel, int ncpus, int nthreads)
 
 	perf_evsel__close_fd(evsel, ncpus, nthreads);
 	perf_evsel__free_fd(evsel);
-	evsel->fd = NULL;
 }
 
 static struct {

Original file line number	Diff line number	Diff line change
`@@ -1081,7 +1081,6 @@ void perf_evsel__close(struct perf_evsel *evsel, int ncpus, int nthreads)`
`1081`	`1081`
`1082`	`1082`	`perf_evsel__close_fd(evsel, ncpus, nthreads);`
`1083`	`1083`	`perf_evsel__free_fd(evsel);`
`1084`		`- evsel->fd = NULL;`
`1085`	`1084`	`}`
`1086`	`1085`
`1087`	`1086`	`static struct {`