Re: [FFmpeg-devel] [PATCH] avformat/mov: Only read the primary item for AVIF

From: Vignesh Venkatasubramanian <vigneshv-at-google.com@ffmpeg.org>
To: James Zern <jzern@google.com>
Cc: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH] avformat/mov: Only read the primary item for AVIF
Date: Wed, 1 Jun 2022 13:38:35 -0700
Message-ID: <CAOJaEP+Ur+V=_7r0+E53qVoqPPd4veLt3AicmHMWTSVERvpbHw@mail.gmail.com> (raw)
In-Reply-To: <CABWgkXKRUvjzhA_KMC+OG+a7P3iYG=xAGyvxeyLJZ+koN6Xijw@mail.gmail.com>

On Wed, Jun 1, 2022 at 10:30 AM James Zern <jzern@google.com> wrote:
>
> On Sun, Apr 24, 2022 at 11:35 AM Vignesh Venkatasubramanian
> <vigneshv-at-google.com@ffmpeg.org> wrote:
> >
> > Update the still AVIF parser to only read the primary item. With this
> > patch, AVIF still images with exif/icc/alpha channel will no longer
> > fail to parse.
> >
> > For example, this patch enables parsing of files in:
> > https://github.com/AOMediaCodec/av1-avif/tree/master/testFiles/Microsoft
> >
>
> Can some of the failing files or similar ones be added to fate?

There are no fate tests for AVIF parsing as of now. I was thinking
about adding some for the muxing. But i am not sure what can be done
here for the parsing.

-- 
Vignesh
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".