[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] audio/mixeng: fix clang 10+ warning
From: |
Gerd Hoffmann |
Subject: |
Re: [PATCH] audio/mixeng: fix clang 10+ warning |
Date: |
Tue, 26 May 2020 07:51:10 +0200 |
On Sat, May 23, 2020 at 10:17:12PM +0200, Volker Rümelin wrote:
> The code in CONV_NATURAL_FLOAT() and CLIP_NATURAL_FLOAT()
> seems to use the constant 2^31-0.5 to convert float to integer
> and back. But the float type lacks the required precision and
> the constant used for the conversion is 2^31. This is equiva-
> lent to a [-1.f, 1.f] <-> [INT32_MIN, INT32_MAX + 1] mapping.
>
> This patch explicitly writes down the used constant. The
> compiler generated code doesn't change.
>
> The constant 2^31 has an exact float representation and the
> clang 10 compiler stops complaining about an implicit int to
> float conversion with a changed value.
>
> A few notes:
> - The conversion of 1.f to INT32_MAX + 1 doesn't overflow. The
> type of the destination variable is int64_t.
> - At a later stage one of the clip_* functions in
> audio/mixeng_template.h limits INT32_MAX + 1 to the integer
> range.
> - The clip_natural_float_* functions in audio/mixeng.c convert
> INT32_MAX and INT32_MAX + 1 to 1.f.
>
> Buglink: https://bugs.launchpad.net/bugs/1878627
> Signed-off-by: Volker Rümelin <address@hidden>
Added to audio queue.
thanks,
Gerd