Commit a6bee71f authored by Max Kellermann's avatar Max Kellermann

decoder/flac: pass audio_format.bits to flac_convert()

Don't use audio_format_sample_size() for identifying the sample format.
parent e0e6813a
...@@ -292,12 +292,12 @@ flac_convert_8(int8_t *dest, ...@@ -292,12 +292,12 @@ flac_convert_8(int8_t *dest,
static void flac_convert(unsigned char *dest, static void flac_convert(unsigned char *dest,
unsigned int num_channels, unsigned int num_channels,
unsigned int bytes_per_sample, unsigned sample_format,
const FLAC__int32 * const buf[], const FLAC__int32 * const buf[],
unsigned int position, unsigned int end) unsigned int position, unsigned int end)
{ {
switch (bytes_per_sample) { switch (sample_format) {
case 2: case 16:
if (num_channels == 2) if (num_channels == 2)
flac_convert_stereo16((int16_t*)dest, buf, flac_convert_stereo16((int16_t*)dest, buf,
position, end); position, end);
...@@ -306,12 +306,13 @@ static void flac_convert(unsigned char *dest, ...@@ -306,12 +306,13 @@ static void flac_convert(unsigned char *dest,
position, end); position, end);
break; break;
case 4: case 24:
case 32:
flac_convert_32((int32_t*)dest, num_channels, buf, flac_convert_32((int32_t*)dest, num_channels, buf,
position, end); position, end);
break; break;
case 1: case 8:
flac_convert_8((int8_t*)dest, num_channels, buf, flac_convert_8((int8_t*)dest, num_channels, buf,
position, end); position, end);
break; break;
...@@ -323,6 +324,7 @@ flac_common_write(struct flac_data *data, const FLAC__Frame * frame, ...@@ -323,6 +324,7 @@ flac_common_write(struct flac_data *data, const FLAC__Frame * frame,
const FLAC__int32 *const buf[]) const FLAC__int32 *const buf[])
{ {
unsigned int c_samp; unsigned int c_samp;
const unsigned sample_format = data->audio_format.bits;
const unsigned int num_channels = frame->header.channels; const unsigned int num_channels = frame->header.channels;
const unsigned int bytes_per_sample = const unsigned int bytes_per_sample =
audio_format_sample_size(&data->audio_format); audio_format_sample_size(&data->audio_format);
...@@ -332,11 +334,6 @@ flac_common_write(struct flac_data *data, const FLAC__Frame * frame, ...@@ -332,11 +334,6 @@ flac_common_write(struct flac_data *data, const FLAC__Frame * frame,
unsigned int num_samples; unsigned int num_samples;
enum decoder_command cmd; enum decoder_command cmd;
if (bytes_per_sample != 1 && bytes_per_sample != 2 &&
bytes_per_sample != 4)
/* exotic unsupported bit rate */
return FLAC__STREAM_DECODER_WRITE_STATUS_ABORT;
for (c_samp = 0; c_samp < frame->header.blocksize; for (c_samp = 0; c_samp < frame->header.blocksize;
c_samp += num_samples) { c_samp += num_samples) {
num_samples = frame->header.blocksize - c_samp; num_samples = frame->header.blocksize - c_samp;
...@@ -344,7 +341,7 @@ flac_common_write(struct flac_data *data, const FLAC__Frame * frame, ...@@ -344,7 +341,7 @@ flac_common_write(struct flac_data *data, const FLAC__Frame * frame,
num_samples = max_samples; num_samples = max_samples;
flac_convert(data->chunk, flac_convert(data->chunk,
num_channels, bytes_per_sample, buf, num_channels, sample_format, buf,
c_samp, c_samp + num_samples); c_samp, c_samp + num_samples);
cmd = decoder_data(data->decoder, data->input_stream, cmd = decoder_data(data->decoder, data->input_stream,
......
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