diff options
author | Adrián Medraño Calvo <adrian@medranocalvo.com> | 2017-07-28 06:54:06 (GMT) |
---|---|---|
committer | Adrián Medraño Calvo <adrian@medranocalvo.com> | 2017-07-28 07:11:19 (GMT) |
commit | d2c078754fbaee1bd22d77202f9af3d706f45b94 (patch) | |
tree | de4a975872eca1b01fc9ac74f175af98e7283ee5 /generic/tkbltGrMarkerLine.C | |
parent | e8635888a9d67eff399ad5d5fdded50dcc7e843a (diff) | |
download | blt-d2c078754fbaee1bd22d77202f9af3d706f45b94.zip blt-d2c078754fbaee1bd22d77202f9af3d706f45b94.tar.gz blt-d2c078754fbaee1bd22d77202f9af3d706f45b94.tar.bz2 |
Stop checking for NULL before free
The 'delete' operator checks whether the lvalue is NULL before freeing.
Diffstat (limited to 'generic/tkbltGrMarkerLine.C')
-rw-r--r-- | generic/tkbltGrMarkerLine.C | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/generic/tkbltGrMarkerLine.C b/generic/tkbltGrMarkerLine.C index 954d4dc..82c9ab8 100644 --- a/generic/tkbltGrMarkerLine.C +++ b/generic/tkbltGrMarkerLine.C @@ -103,8 +103,7 @@ LineMarker::~LineMarker() { if (gc_) graphPtr_->freePrivateGC(gc_); - if (segments_) - delete [] segments_; + delete [] segments_; } int LineMarker::configure() @@ -151,11 +150,9 @@ void LineMarker::map() { LineMarkerOptions* ops = (LineMarkerOptions*)ops_; + delete [] segments_; + segments_ = NULL; nSegments_ = 0; - if (segments_) { - delete [] segments_; - segments_ = NULL; - } if (!ops->worldPts || (ops->worldPts->num < 2)) return; |