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 /src/cpu/abstract_cpu.cpp | |
| 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>
Diffstat (limited to 'src/cpu/abstract_cpu.cpp')
| -rw-r--r-- | src/cpu/abstract_cpu.cpp | 6 |
1 files changed, 2 insertions, 4 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(); } |