[FFmpeg-devel] [PATCH] [libavutil] Add saturated add/sub operations for int64_t.

[James Almer]

On 5/1/2020 6:36 AM, Carl Eugen Hoyos wrote:
> Am Fr., 1. Mai 2020 um 01:31 Uhr schrieb James Almer <jamrial at gmail.com>:
>>
>> On 4/30/2020 8:20 PM, Carl Eugen Hoyos wrote:
>>> Am Fr., 1. Mai 2020 um 00:20 Uhr schrieb Dale Curtis <dalecurtis at chromium.org>:
>>>>
>>>> Many places are using their own custom code for handling overflow
>>>> around timestamps or other int64_t values. There are enough of these
>>>> now that having some common saturated math functions seems sound.
>>>
>>>> This adds implementations that just use the builtin functions for
>>>> recent gcc, clang when available or implements its own version for
>>>> older compilers.
>>>
>>> Should be tested in configure and I believe this should be a separate patch.
>>
>> Not really, the AV_GCC_VERSION_AT_LEAST helper macro exists precisely
>> for this purpose.
> 
> The macro exists to avoid separate patches?

No, it exists to not require configure checks just to enable a path for
gcc/clang and another for other compilers.

> 
> Carl Eugen
> _______________________________________________
> 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".
>