[FFmpeg-devel] [PATCH] avformat/mov: add VP8 codec support

James Almer jamrial at gmail.com
Wed Feb 7 04:12:59 EET 2018


On 2/6/2018 9:25 PM, Carl Eugen Hoyos wrote:
> 2018-02-06 2:39 GMT+01:00 James Almer <jamrial at gmail.com>:
>> Demuxing only. Muxing is disabled as altref frame handling is not
>> defined, and there's no way to know the presence of such frames
>> during stream initialization.
> 
> strict -2?

Allowing muxing something that's explicitly listed as not supported by
the current spec will come bite us at a later point, so I'd rather not.

> 
>> +            if (track->par->codec_id == AV_CODEC_ID_VP8) {
>> +                /* altref frames handling is not defined in the spec as of version v1.0,
>> +                 * so just forbid muxing VP8 streams altogether */
>> +                av_log(s, AV_LOG_ERROR, "VP8 muxing is currently not supported.\n");
>> +                return AVERROR(EINVAL);
> 
> Could be PATCHWELCOME.

Alright.

> 
> Feel free to push, thank you, Carl Eugen

Pushed, thanks.


More information about the ffmpeg-devel mailing list