From f2a75fbfc77b9f6d7118accb42391bef85f0dfcf Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sat, 23 Aug 2014 13:57:57 +0200 Subject: decoder/dsf: refactor the main decoder loop Check for STOP before decoding the first chunk. This reduces the command latency. --- src/decoder/plugins/DsfDecoderPlugin.cxx | 26 +++++++------------------- 1 file changed, 7 insertions(+), 19 deletions(-) (limited to 'src/decoder/plugins/DsfDecoderPlugin.cxx') diff --git a/src/decoder/plugins/DsfDecoderPlugin.cxx b/src/decoder/plugins/DsfDecoderPlugin.cxx index 3b967fc14..2e8bc4f2a 100644 --- a/src/decoder/plugins/DsfDecoderPlugin.cxx +++ b/src/decoder/plugins/DsfDecoderPlugin.cxx @@ -254,7 +254,8 @@ dsf_decode_chunk(Decoder &decoder, InputStream &is, { const size_t block_size = channels * DSF_BLOCK_SIZE; - for (offset_type i = 0; i < n_blocks;) { + auto cmd = decoder_get_command(decoder); + for (offset_type i = 0; i < n_blocks && cmd != DecoderCommand::STOP;) { /* worst-case buffer size */ uint8_t buffer[MAX_CHANNELS * DSF_BLOCK_SIZE]; if (!decoder_read_full(&decoder, is, buffer, block_size)) @@ -266,25 +267,12 @@ dsf_decode_chunk(Decoder &decoder, InputStream &is, uint8_t interleaved_buffer[MAX_CHANNELS * DSF_BLOCK_SIZE]; InterleaveDsfBlock(interleaved_buffer, buffer, channels); - const auto cmd = decoder_data(decoder, is, - interleaved_buffer, block_size, - sample_rate / 1000); - switch (cmd) { - case DecoderCommand::NONE: - ++i; - break; - - case DecoderCommand::START: - case DecoderCommand::STOP: - return false; - - case DecoderCommand::SEEK: - - /* not implemented yet */ - decoder_seek_error(decoder); - break; - } + cmd = decoder_data(decoder, is, + interleaved_buffer, block_size, + sample_rate / 1000); + ++i; } + return true; } -- cgit v1.2.3