> On Jan 3, 2020, at 12:20 AM, Nicolas George <geo...@nsup.org> wrote: > > Zhao Zhili (12020-01-03): >> Ping for review, thanks! I'm planning to use these API in ff_all_formats() >> if the patch is acceptable. > > For ff_all_formats(), as I told you, you cannot assume there is no gap: > if you want to optimize it, just make a first loop to count the slots > needed.
The comment for AV_PIX_FMT_NB makes me wonder maybe there is requirement of such API. After search and fixed the use of AV_PIX_FMT_NB in other library other than libavutil, most of them don't need AV_PIX_FMT_NB actually. Please ignore the patch if such API are useless. > > Regards, > > -- > Nicolas George > > _______________________________________________ > 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". _______________________________________________ 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".
