On 2/6/2018 9:25 PM, Carl Eugen Hoyos wrote:
> 2018-02-06 2:39 GMT+01:00 James Almer <jamr...@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.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
