shithub: openh264

Info  •  Files  •  Log  •  Branches

Remove an unnecessary cast when setting function pointers

By removing the casts we make sure that the function signature
actually matches and isn't silenced by the cast.

--- a/codec/encoder/core/src/encoder.cpp

+++ b/codec/encoder/core/src/encoder.cpp

@@ -150,11 +150,11 @@

 void WelsInitBGDFunc (SWelsFuncPtrList* pFuncList, const bool_t kbEnableBackgroundDetection) {

   if (kbEnableBackgroundDetection) {

-    pFuncList->pfInterMdBackgroundDecision = (PInterMdBackgroundDecisionFunc)WelsMdInterJudgeBGDPskip;

-    pFuncList->pfInterMdBackgroundInfoUpdate = (PInterMdBackgroundInfoUpdateFunc)WelsMdInterUpdateBGDInfo;

+    pFuncList->pfInterMdBackgroundDecision = WelsMdInterJudgeBGDPskip;

+    pFuncList->pfInterMdBackgroundInfoUpdate = WelsMdInterUpdateBGDInfo;

   } else {

-    pFuncList->pfInterMdBackgroundDecision = (PInterMdBackgroundDecisionFunc)WelsMdInterJudgeBGDPskipFalse;

-    pFuncList->pfInterMdBackgroundInfoUpdate = (PInterMdBackgroundInfoUpdateFunc)WelsMdInterUpdateBGDInfoNULL;

+    pFuncList->pfInterMdBackgroundDecision = WelsMdInterJudgeBGDPskipFalse;

+    pFuncList->pfInterMdBackgroundInfoUpdate = WelsMdInterUpdateBGDInfoNULL;

   }

 }