[FFmpeg-devel] [PATCH] avformat/movenc: forbid muxing AV1 streams until the spec is finished

Tomas Härdin tjoppen at acc.umu.se
Mon Apr 16 10:57:48 EEST 2018


sön 2018-04-15 klockan 21:40 -0300 skrev James Almer:
> This prevents creating potentially broken files, as both the AV1 and
> the AV1 in ISOMBFF specs are unfinished.
> 
> > Signed-off-by: James Almer <jamrial at gmail.com>
> ---
> This is mainly so it's effective in ffmpeg 4.0, the first version to
> support libaom (including pre-bitstream freeze builds). Best to
> prevent muxing files that may not work anymore a few months from now.
> 
>  libavformat/movenc.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/libavformat/movenc.c b/libavformat/movenc.c
> index d03d7906a1..dc5c842cb8 100644
> --- a/libavformat/movenc.c
> +++ b/libavformat/movenc.c
> @@ -6081,6 +6081,10 @@ static int mov_init(AVFormatContext *s)
>                      av_log(s, AV_LOG_ERROR, "VP9 only supported in MP4.\n");
>                      return AVERROR(EINVAL);
>                  }
> +            } else if (track->par->codec_id == AV_CODEC_ID_AV1) {
> +                /* spec is not finished, so forbid for now */
> +                av_log(s, AV_LOG_ERROR, "AV1 muxing is currently not supported.\n");
> +                return AVERROR_PATCHWELCOME;

Looks OK, we shouldn't produce broken files of course

/Tomas


More information about the ffmpeg-devel mailing list