[FFmpeg-devel] [PATCH] avcodec/vvcdec: decode extradata to support container formats

[Nuo Mi]

On Sat, Jan 6, 2024 at 10:10 AM Nuo Mi <nuomi2021 at gmail.com> wrote:

>
>
> On Sat, Jan 6, 2024 at 9:32 AM James Almer <jamrial at gmail.com> wrote:
>
>> On 1/5/2024 10:27 PM, Nuo Mi wrote:
>> > On Sat, Jan 6, 2024 at 9:25 AM Nuo Mi <nuomi2021 at gmail.com> wrote:
>> >
>> >> For example:
>> >> wget
>> >>
>> https://www.elecard.com/storage/video/NovosobornayaSquare_1920x1080.mp4
>> >> ./ffplay NovosobornayaSquare_1920x1080.mp4
>> >> ---
>> >>   libavcodec/vvc/vvcdec.c | 6 ++++++
>> >>   1 file changed, 6 insertions(+)
>> >>
>> >> diff --git a/libavcodec/vvc/vvcdec.c b/libavcodec/vvc/vvcdec.c
>> >> index 608d497cb7..54ada28124 100644
>> >> --- a/libavcodec/vvc/vvcdec.c
>> >> +++ b/libavcodec/vvc/vvcdec.c
>> >> @@ -977,6 +977,12 @@ static av_cold int vvc_decode_init(AVCodecContext
>> >> *avctx)
>> >>       if (ret)
>> >>           return ret;
>> >>
>> >> +    if (avctx->extradata_size > 0 && avctx->extradata) {
>> >> +        ret = ff_cbs_read_extradata_from_codec(s->cbc,
>> &s->current_frame,
>> >> avctx);
>> >> +        if (ret < 0)
>> >> +            return ret;
>> >> +    }
>> >> +
>> >>       s->nb_fcs = (avctx->flags & AV_CODEC_FLAG_LOW_DELAY) ? 1 :
>> delayed;
>> >>       s->fcs = av_calloc(s->nb_fcs, sizeof(*s->fcs));
>> >>       if (!s->fcs)
>> >> --
>> >> 2.25.1
>>
>> LGTM
>>
>
>> >>
>> >> Depends on
>> >
>> https://patchwork.ffmpeg.org/project/ffmpeg/patch/20231103095720.32426-3-thomas.ff@spin-digital.com/
>>
>> I don't see why this depends on that patch. Decoder and muxer are two
>> separate things.
>
> Thank you. I'll push it tomorrow if there are no objections.
>
Pushed.

> Yes, we can push it separately, but the user needs the demux patch to
> decode MP4 files. Just want to call out it :)
>
>>
>
> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>>
>