[FFmpeg-devel] IMF demuxer ping

Anton Khirnov anton at khirnov.net
Mon Dec 6 12:16:17 EET 2021


Quoting Lynne (2021-12-05 11:13:17)
> 5 Dec 2021, 02:33 by pal at sandflow.com:
> 
> > Hi all,
> >
> > Quick ping re: libavformat/imf demuxer patch set.
> >
> > All outstanding feedback (thanks!) has been addressed as far as I know.
> >
> >  What are the next steps?
> >
> > It would be good to make progress while it is fresh in peoples' minds.
> >
> > Latest patchset at [1] and tracking PR at [2] (including memory leak checking):
> >
> > [1] http://ffmpeg.org/pipermail/ffmpeg-devel/2021-November/288173.html
> > [2] https://github.com/sandflow/ffmpeg-imf/pull/74
> >
> > Looking forward to (finally) adding support for IMF demuxing to ffmpeg.
> >
> 
> There are still many issues.
> Put your custom header beneath the standard ffmpeg header. Or
> remove it altogether if you're allowed to.
> You're not following our code style at all:
> All internal functions (except intra-file functions) must have
> an ff_<name> prefix. If it's a struct, the prefix is FF<name>.

actually we do not require any prefixing for internal structs, since
they are not visible in the ABI.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list