diff options
author | SF Markus Elfring <elfring@users.sourceforge.net> | 2014-10-19 00:50:29 +0200 |
---|---|---|
committer | Alexandra Yates <alexandra.yates@linux.intel.com> | 2014-10-30 20:05:53 -0700 |
commit | fb6f65a01eca950e2e12a87492268e7d2105aa0b (patch) | |
tree | dd4faa21f63685e395350aeff6d4ef9376ac34b7 | |
parent | 9716455a28bea6f59ba9081479e431f6f9548fba (diff) |
Deletion of unnecessary checks before specific function calls
The function "free" is documented in the way that no action shall occur for
a passed null pointer. It is therefore not needed that a function caller
repeats a corresponding check.
http://stackoverflow.com/questions/18775608/free-a-null-pointer-anyway-or-check-first
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
-rw-r--r-- | src/cpu/abstract_cpu.cpp | 6 | ||||
-rw-r--r-- | src/devlist.cpp | 9 | ||||
-rw-r--r-- | src/perf/perf.cpp | 6 | ||||
-rw-r--r-- | src/report/report-maker.cpp | 6 | ||||
-rw-r--r-- | traceevent/event-parse.c | 3 |
5 files changed, 10 insertions, 20 deletions
diff --git a/src/cpu/abstract_cpu.cpp b/src/cpu/abstract_cpu.cpp index 3b841bf..a3a9ffa 100644 --- a/src/cpu/abstract_cpu.cpp +++ b/src/cpu/abstract_cpu.cpp @@ -34,14 +34,12 @@ abstract_cpu::~abstract_cpu() { unsigned int i=0; for (i=0; i < cstates.size(); i++){ - if(cstates[i]) - delete cstates[i]; + delete cstates[i]; } cstates.clear(); for (i=0; i < pstates.size(); i++){ - if(pstates[i]) - delete pstates[i]; + delete pstates[i]; } pstates.clear(); } diff --git a/src/devlist.cpp b/src/devlist.cpp index 20fe5da..7f599ac 100644 --- a/src/devlist.cpp +++ b/src/devlist.cpp @@ -77,18 +77,15 @@ void clean_open_devices() unsigned int i=0; for (i = 0; i < one.size(); i++) { - if(one[i]) - free(one[i]); + free(one[i]); } for (i = 0; i < two.size(); i++) { - if(two[i]) - free(two[i]); + free(two[i]); } for (i = 0; i < devpower.size(); i++){ - if(devpower[i]) - free(devpower[i]); + free(devpower[i]); } } diff --git a/src/perf/perf.cpp b/src/perf/perf.cpp index b176256..b1103a7 100644 --- a/src/perf/perf.cpp +++ b/src/perf/perf.cpp @@ -149,8 +149,7 @@ void perf_event::create_perf_event(char *eventname, int _cpu) void perf_event::set_event_name(const char *event_name) { - if (name) - free(name); + free(name); name = strdup(event_name); if (!name) { fprintf(stderr, "failed to allocate event name\n"); @@ -168,8 +167,7 @@ void perf_event::set_event_name(const char *event_name) perf_event::~perf_event(void) { - if (name) - free(name); + free(name); if (perf_event::pevent->ref_count == 1) { pevent_free(perf_event::pevent); diff --git a/src/report/report-maker.cpp b/src/report/report-maker.cpp index f2fe522..4049a54 100644 --- a/src/report/report-maker.cpp +++ b/src/report/report-maker.cpp @@ -47,8 +47,7 @@ report_maker::report_maker(report_type t) report_maker::~report_maker() { - if (formatter) - delete formatter; + delete formatter; } /* ************************************************************************ */ @@ -98,8 +97,7 @@ report_maker::set_type(report_type t) void report_maker::setup_report_formatter() { - if (formatter) - delete formatter; + delete formatter; if (type == REPORT_HTML) formatter = new report_formatter_html(); diff --git a/traceevent/event-parse.c b/traceevent/event-parse.c index 6195f6f..5a717a0 100644 --- a/traceevent/event-parse.c +++ b/traceevent/event-parse.c @@ -1023,8 +1023,7 @@ static enum event_type force_token(const char *str, char **tok) static void free_token(char *tok) { - if (tok) - free(tok); + free(tok); } static enum event_type read_token(char **tok) |