[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