Commit a6838919 authored by Rémi Denis-Courmont's avatar Rémi Denis-Courmont

aout: always use S16N (rather than FI32) in absence of FPU

None of the real audio outputs support FI32; they already typically
converted down to S16 internally. Moreover, mixing FI32 requires
64-bits (signed) multiplication, which non-FPU processors typically
do not support natively.
parent bb94e7b5
...@@ -234,11 +234,6 @@ int aout_OutputNew( audio_output_t *p_aout, ...@@ -234,11 +234,6 @@ int aout_OutputNew( audio_output_t *p_aout,
if( HAVE_FPU ) if( HAVE_FPU )
owner->mixer_format.i_format = VLC_CODEC_FL32; owner->mixer_format.i_format = VLC_CODEC_FL32;
else else
/* Otherwise, audio filters will not work. Use fixed-point if the input has
* more than 16-bits depth. */
if( p_format->i_bitspersample > 16 || !AOUT_FMT_LINEAR(p_format))
owner->mixer_format.i_format = VLC_CODEC_FI32;
else
/* Fallback to 16-bits. This avoids pointless conversion to and from /* Fallback to 16-bits. This avoids pointless conversion to and from
* 32-bits samples for the sole purpose of software mixing. */ * 32-bits samples for the sole purpose of software mixing. */
owner->mixer_format.i_format = VLC_CODEC_S16N; owner->mixer_format.i_format = VLC_CODEC_S16N;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment