[FFmpeg-devel] [PATCH] avutil/frame: change the type of the crop fields to unsigned in the next major bump

[Marton Balint]

On Mon, 17 Apr 2023, Anton Khirnov wrote:

> Quoting James Almer (2023-04-17 17:34:08)
>> size_t is unnecessarily big on 64bit arches when width and height are ints.
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavutil/frame.h   | 7 +++++++
>>  libavutil/version.h | 1 +
>>  2 files changed, 8 insertions(+)
>>
>> diff --git a/libavutil/frame.h b/libavutil/frame.h
>> index f85d630c5c..e10a1c65e3 100644
>> --- a/libavutil/frame.h
>> +++ b/libavutil/frame.h
>> @@ -722,10 +722,17 @@ typedef struct AVFrame {
>>       * the frame intended for presentation.
>>       * @{
>>       */
>> +#if FF_API_CROP_SIZE_T
>>      size_t crop_top;
>>      size_t crop_bottom;
>>      size_t crop_left;
>>      size_t crop_right;
>> +#else
>> +    unsigned int crop_top;
>> +    unsigned int crop_bottom;
>> +    unsigned int crop_left;
>> +    unsigned int crop_right;
>> +#endif
>
> Ok, but should be mentioned in APIchanges

I don't oppose this, but isn't this the same kind of change I wanted to do 
for frame_number which was rejected because of e.g. printf format string 
concerns? So based on that logic the proper way is to introduce new fields 
with different types and deprecate the old ones. If we want to be 
consistent with our API rules...

Regards,
Marton