Anton Khirnov:
> 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.
>
Correct. Will fix.
- Andreas
_______________________________________________
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".
