Re: [FFmpeg-devel] [PATCH v5] avformat: add MMTP parser and MMT/TLV demuxer

2023-05-02 Thread SuperFashi
I can remove the NULL checks, sure. I could change L289 in mmttlv.c to a freep and reset the priv_data. Other places have no need to change. On Tue, May 2, 2023 at 9:58 PM Paul B Mahol wrote: > Use av_freep(), non need to check for NULL before av_freep() / av_free() >

Re: [FFmpeg-devel] [PATCH v5] avformat: add MMTP parser and MMT/TLV demuxer

2023-05-02 Thread Paul B Mahol
Use av_freep(), non need to check for NULL before av_freep() / av_free() ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org

[FFmpeg-devel] [PATCH v5] avformat: add MMTP parser and MMT/TLV demuxer

2023-05-01 Thread SuperFashi
v1 -> v2: Refactor using GetByteContext; Fix compile error. v2 -> v3: Remove debug statement. v3 -> v4: Squash commits v4 -> v5: Improve portability; Cosmetic changes. This patch adds an MPEG Media Transport Protocol (MMTP) parser, as defined in ISO/IEC 23008-1, and an MMT protocol over TLV