vlc | branch: master | Marvin Scholz <epira...@gmail.com> | Fri Jul  3 02:18:56 
2020 +0200| [4e843d37043790671301eb829477121dc94a406d] | committer: Marvin 
Scholz

videotoolbox: remove useless CopyDecoderExtradataDefault

Calling ExtradataInfoCreate with NULL for the data argument makes it
return NULL anyway, so instead just do not provide this callback at all
as the code already handles it fine when this callback is NULL.

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=4e843d37043790671301eb829477121dc94a406d
---

 modules/codec/videotoolbox.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/modules/codec/videotoolbox.c b/modules/codec/videotoolbox.c
index 75229ad722..709f2db4f3 100644
--- a/modules/codec/videotoolbox.c
+++ b/modules/codec/videotoolbox.c
@@ -785,12 +785,6 @@ static CFDictionaryRef CopyDecoderExtradataMPEG4(decoder_t 
*p_dec)
         return NULL; /* MPEG4 without esds ? */
 }
 
-static CFDictionaryRef CopyDecoderExtradataDefault(decoder_t *p_dec)
-{
-    VLC_UNUSED(p_dec);
-    return ExtradataInfoCreate(NULL, NULL, 0); /* Empty Needed ? */
-}
-
 /* !Codec Specific */
 
 static void InsertIntoDPB(decoder_sys_t *p_sys, frame_info_t *p_info)
@@ -1455,7 +1449,7 @@ static int OpenDecoder(vlc_object_t *p_this)
             break;
 
         default:
-            p_sys->pf_copy_extradata = CopyDecoderExtradataDefault;
+            p_sys->pf_copy_extradata = NULL;
             break;
     }
 

_______________________________________________
vlc-commits mailing list
vlc-commits@videolan.org
https://mailman.videolan.org/listinfo/vlc-commits

Reply via email to