Quoting Andreas Rheinhardt (2022-09-26 21:58:55)
> Instead use av_pix_fmt_desc_next(). It is still possible
> to check its return values by comparing it with the
> (currently) expected values and the code does so.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@outlook.com>
> ---
> libavutil/pixdesc.c | 17 ++++++++++++-----
> 1 file changed, 12 insertions(+), 5 deletions(-)
>
> diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c
> index c42a0242c5..f6755f41df 100644
> --- a/libavutil/pixdesc.c
> +++ b/libavutil/pixdesc.c
> @@ -2913,10 +2913,10 @@ int av_pix_fmt_count_planes(enum AVPixelFormat
> pix_fmt)
> }
>
> void ff_check_pixfmt_descriptors(void){
> - int i, j;
> + const AVPixFmtDescriptor *d, *last = NULL;
> + int i;
>
> - for (i=0; i<FF_ARRAY_ELEMS(av_pix_fmt_descriptors); i++) {
> - const AVPixFmtDescriptor *d = &av_pix_fmt_descriptors[i];
> + for (i = AV_PIX_FMT_NONE, d = NULL; i++, d = av_pix_fmt_desc_next(d);) {
> uint8_t fill[4][8+6+3] = {{0}};
> uint8_t *data[4] = {fill[0], fill[1], fill[2], fill[3]};
> int linesize[4] = {0,0,0,0};
> @@ -2927,9 +2927,15 @@ void ff_check_pixfmt_descriptors(void){
> av_assert0(d->log2_chroma_w <= 3);
> av_assert0(d->log2_chroma_h <= 3);
> av_assert0(d->nb_components <= 4);
> - av_assert2(av_get_pix_fmt(d->name) == i);
> + av_assert2(av_get_pix_fmt(d->name) == av_pix_fmt_desc_get_id(d));
> +
> + /* The following two checks as well as the one after the loop
> + * would need to be changed if we changed the way the descriptors
> + * are stored. */
> + av_assert0(i == av_pix_fmt_desc_get_id(d));
> + av_assert0(!last || last + 1 == d);
Don't see last being set.
--
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".
