shithub: openh264

Info  •  Files  •  Log  •  Branches

Remove unnecesary use of _SafeDelete

It is not necessary to check whether a pointer is NULL before
deleting it, "delete NULL" is explicitly ok.

The deleted pointer is a local variable here, so setting it to
NULL after deleting isn't necessary in these cases either.

--- a/codec/processing/src/common/WelsFrameWork.cpp

+++ b/codec/processing/src/common/WelsFrameWork.cpp

@@ -79,7 +79,7 @@

 }

 EResult DestroySpecificVpInterface (IWelsVP* pCtx) {

-  _SafeDelete (pCtx);

+  delete pCtx;

   return RET_SUCCESS;

 }

--- a/codec/processing/src/common/WelsFrameWorkEx.cpp

+++ b/codec/processing/src/common/WelsFrameWorkEx.cpp

@@ -87,7 +87,7 @@

 EResult DestroySpecificVpInterface (IWelsVPc* pCtx) {

   if (pCtx) {

     DestroySpecificVpInterface (WelsStaticCast (IWelsVP*, pCtx->pCtx));

-    _SafeDelete (pCtx);

+    delete pCtx;

   }

   return RET_SUCCESS;