diff options
| author | Dan McGee <dan@archlinux.org> | 2009-01-25 12:52:37 +0100 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2009-01-25 12:52:37 +0100 |
| commit | 27baf6913ea2415300c8d6650773e5eb10a01943 (patch) | |
| tree | c56c322022e91b0e096cb0f29d76685e0a571262 /src/decoder/mp4_plugin.c | |
| parent | 16796b1209dc1070c08c4a5996d691aa289a4481 (diff) | |
| download | mpd-27baf6913ea2415300c8d6650773e5eb10a01943.tar.gz mpd-27baf6913ea2415300c8d6650773e5eb10a01943.tar.xz mpd-27baf6913ea2415300c8d6650773e5eb10a01943.zip | |
alsa: fix option parsing and restore default period_time
Two bugs here led to a large number of interrupts being generated on the
sound card when ALSA output is being used. Because we specify no default
period_time, the sound card gives us 3000 interrupts/sec rather than a more
sane 20 or 30. This completes the revert of dd7711 already started by
4ca24f.
The larger bug was in the change to config_get_block_unsigned() and using 0
as the default value for both 'buffer_time' and 'period_time'. This means
any pre-setting of these options in newAlsaData() gets wiped out. Add a new
default for period_time, and ensure default values for buffer_time and
period_time are used if none are provided by the user.
Signed-off-by: Dan McGee <dan@archlinux.org>
[mk: set defaults in newAlsaData() to fix auto-configuration; renamed
"_MS" back to "_US" because ALSA expects microseconds, not milliseconds]
Signed-off-by: Max Kellermann <max@duempel.org>
Diffstat (limited to 'src/decoder/mp4_plugin.c')
0 files changed, 0 insertions, 0 deletions