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/tkbltGrElemOption.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/tkbltGrElemOption.C')
| -rw-r--r-- | generic/tkbltGrElemOption.C | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/generic/tkbltGrElemOption.C b/generic/tkbltGrElemOption.C index 45591ac..a0a67e6 100644 --- a/generic/tkbltGrElemOption.C +++ b/generic/tkbltGrElemOption.C @@ -136,8 +136,7 @@ static Tcl_Obj* ValuesGetProc(ClientData clientData, Tk_Window tkwin, static void ValuesFreeProc(ClientData clientData, Tk_Window tkwin, char *ptr) { ElemValues* valuesPtr = *(ElemValues**)ptr; - if (valuesPtr) - delete valuesPtr; + delete valuesPtr; } static Tk_CustomOptionSetProc PairsSetProc; @@ -171,12 +170,10 @@ static int PairsSetProc(ClientData clientData, Tcl_Interp* interp, } nValues /= 2; - if (coordsPtr->x) - delete coordsPtr->x; + delete coordsPtr->x; coordsPtr->x = new ElemValuesSource(nValues); - if (coordsPtr->y) - delete coordsPtr->y; + delete coordsPtr->y; coordsPtr->y = new ElemValuesSource(nValues); int ii=0; |