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

[Vignesh Venkatasubramanian]

On Thu, Jun 2, 2022 at 1:35 PM James Zern <jzern at google.com> wrote:
>
> On Wed, Jun 1, 2022 at 1:38 PM Vignesh Venkatasubramanian
> <vigneshv at google.com> wrote:
> >
> > On Wed, Jun 1, 2022 at 10:30 AM James Zern <jzern at google.com> wrote:
> > >
> > > On Sun, Apr 24, 2022 at 11:35 AM Vignesh Venkatasubramanian
> > > <vigneshv-at-google.com at 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.
> >
>
> Thanks. Are there any for general mov/mp4 parsing that could be
> extended? This looks good otherwise.

>From what i see, most of the mov tests only seem to be for muxing. I'm
not entirely certain about how to add a test for AVIF parsing. If
anybody has an idea, i'd be open to adding it.


--
Vignesh