[FFmpeg-devel] [PATCH 1/8] avutil/pixdesc: add av_pix_fmt_total_count and av_sample_fmt_total_count

Zhao Zhili quinkblack at foxmail.com
Thu Jan 2 18:48:56 EET 2020



> On Jan 3, 2020, at 12:20 AM, Nicolas George <george at 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 at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".



More information about the ffmpeg-devel mailing list