Quoting Lynne (2021-12-05 11:13:17) > 5 Dec 2021, 02:33 by p...@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 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
