[FFmpeg-devel] [PATCH 1/5] avcodec/ffv1enc: Fix handling of 32bit unsigned sambols
Alexander Strasser
eclipse7 at gmx.net
Sun Jan 12 14:37:05 EET 2025
On 2025-01-10 05:06 +0100, Michael Niedermayer wrote:
> This may be needed for floats
>
> Sponsored-by: Sovereign Tech Fund
> Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> ---
> libavcodec/ffv1enc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/ffv1enc.c b/libavcodec/ffv1enc.c
> index 1a0413895a1..e2db1b41640 100644
> --- a/libavcodec/ffv1enc.c
> +++ b/libavcodec/ffv1enc.c
> @@ -199,7 +199,7 @@ static av_always_inline av_flatten void put_symbol_inline(RangeCoder *c,
> } while (0)
>
> if (v) {
> - const int a = FFABS(v);
> + const unsigned a = is_signed ? FFABS(v) : v;
> const int e = av_log2(a);
> put_rac(c, state + 0, 0);
> if (e <= 9) {
> --
LGTM but s/sambols/symbols/ in the Subject/Git-short-message I suspect.
Alexander
More information about the ffmpeg-devel
mailing list