From 0a5c991ab57217dbb74e11ca29cc531a7fb3c3ae Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Wed, 23 Oct 2013 23:20:55 +0200 Subject: decoder/wavpack: move variable declarations --- src/decoder/WavpackDecoderPlugin.cxx | 87 ++++++++++++++---------------------- 1 file changed, 34 insertions(+), 53 deletions(-) (limited to 'src/decoder/WavpackDecoderPlugin.cxx') diff --git a/src/decoder/WavpackDecoderPlugin.cxx b/src/decoder/WavpackDecoderPlugin.cxx index 3588f0797..2275e5a91 100644 --- a/src/decoder/WavpackDecoderPlugin.cxx +++ b/src/decoder/WavpackDecoderPlugin.cxx @@ -140,19 +140,13 @@ wavpack_bits_to_sample_format(bool is_float, int bytes_per_sample) static void wavpack_decode(Decoder &decoder, WavpackContext *wpc, bool can_seek) { - bool is_float; - SampleFormat sample_format; - AudioFormat audio_format; - format_samples_t format_samples; - float total_time; - int bytes_per_sample, output_sample_size; - - is_float = (WavpackGetMode(wpc) & MODE_FLOAT) != 0; - sample_format = + bool is_float = (WavpackGetMode(wpc) & MODE_FLOAT) != 0; + SampleFormat sample_format = wavpack_bits_to_sample_format(is_float, WavpackGetBytesPerSample(wpc)); Error error; + AudioFormat audio_format; if (!audio_format_init_checked(audio_format, WavpackGetSampleRate(wpc), sample_format, @@ -161,16 +155,15 @@ wavpack_decode(Decoder &decoder, WavpackContext *wpc, bool can_seek) return; } - if (is_float) { - format_samples = format_samples_float; - } else { - format_samples = format_samples_int; - } + const format_samples_t format_samples = is_float + ? format_samples_float + : format_samples_int; - total_time = WavpackGetNumSamples(wpc); - total_time /= audio_format.sample_rate; - bytes_per_sample = WavpackGetBytesPerSample(wpc); - output_sample_size = audio_format.GetFrameSize(); + const float total_time = float(WavpackGetNumSamples(wpc)) + / audio_format.sample_rate; + + const int bytes_per_sample = WavpackGetBytesPerSample(wpc); + const int output_sample_size = audio_format.GetFrameSize(); /* wavpack gives us all kind of samples in a 32-bit space */ int32_t chunk[1024]; @@ -220,10 +213,7 @@ static bool wavpack_tag_float(WavpackContext *wpc, const char *key, float *value_r) { char buffer[64]; - int ret; - - ret = WavpackGetTagItem(wpc, key, buffer, sizeof(buffer)); - if (ret <= 0) + if (WavpackGetTagItem(wpc, key, buffer, sizeof(buffer)) <= 0) return false; *value_r = atof(buffer); @@ -291,10 +281,8 @@ static bool wavpack_scan_file(const char *fname, const struct tag_handler *handler, void *handler_ctx) { - WavpackContext *wpc; char error[ERRORLEN]; - - wpc = WavpackOpenFileInput(fname, error, OPEN_TAGS, 0); + WavpackContext *wpc = WavpackOpenFileInput(fname, error, OPEN_TAGS, 0); if (wpc == nullptr) { FormatError(wavpack_domain, "failed to open WavPack file \"%s\": %s", @@ -462,11 +450,6 @@ wavpack_open_wvc(Decoder &decoder, const char *uri, Mutex &mutex, Cond &cond, struct wavpack_input *wpi) { - InputStream *is_wvc; - char *wvc_url = nullptr; - char first_byte; - size_t nbytes; - /* * As we use dc->utf8url, this function will be bad for * single files. utf8url is not absolute file path :/ @@ -474,9 +457,10 @@ wavpack_open_wvc(Decoder &decoder, const char *uri, if (uri == nullptr) return nullptr; - wvc_url = g_strconcat(uri, "c", nullptr); + char *wvc_url = g_strconcat(uri, "c", nullptr); - is_wvc = InputStream::Open(wvc_url, mutex, cond, IgnoreError()); + InputStream *is_wvc = InputStream::Open(wvc_url, mutex, cond, + IgnoreError()); g_free(wvc_url); if (is_wvc == nullptr) @@ -486,9 +470,9 @@ wavpack_open_wvc(Decoder &decoder, const char *uri, * And we try to buffer in order to get know * about a possible 404 error. */ - nbytes = decoder_read( - decoder, *is_wvc, &first_byte, sizeof(first_byte) - ); + char first_byte; + size_t nbytes = decoder_read(decoder, *is_wvc, + &first_byte, sizeof(first_byte)); if (nbytes == 0) { is_wvc->Close(); return nullptr; @@ -506,16 +490,13 @@ wavpack_open_wvc(Decoder &decoder, const char *uri, static void wavpack_streamdecode(Decoder &decoder, InputStream &is) { - char error[ERRORLEN]; - WavpackContext *wpc; - InputStream *is_wvc; int open_flags = OPEN_NORMALIZE; - struct wavpack_input isp, isp_wvc; bool can_seek = is.seekable; - is_wvc = wavpack_open_wvc(decoder, is.uri.c_str(), - is.mutex, is.cond, - &isp_wvc); + wavpack_input isp_wvc; + InputStream *is_wvc = wavpack_open_wvc(decoder, is.uri.c_str(), + is.mutex, is.cond, + &isp_wvc); if (is_wvc != nullptr) { open_flags |= OPEN_WVC; can_seek &= is_wvc->seekable; @@ -525,12 +506,15 @@ wavpack_streamdecode(Decoder &decoder, InputStream &is) open_flags |= OPEN_STREAMING; } + wavpack_input isp; wavpack_input_init(&isp, decoder, is); - wpc = WavpackOpenFileInputEx( - &mpd_is_reader, &isp, - open_flags & OPEN_WVC ? &isp_wvc : nullptr, - error, open_flags, 23 - ); + + char error[ERRORLEN]; + WavpackContext *wpc = + WavpackOpenFileInputEx(&mpd_is_reader, &isp, + open_flags & OPEN_WVC + ? &isp_wvc : nullptr, + error, open_flags, 23); if (wpc == nullptr) { FormatError(wavpack_domain, @@ -553,12 +537,9 @@ static void wavpack_filedecode(Decoder &decoder, const char *fname) { char error[ERRORLEN]; - WavpackContext *wpc; - - wpc = WavpackOpenFileInput( - fname, error, - OPEN_TAGS | OPEN_WVC | OPEN_NORMALIZE, 23 - ); + WavpackContext *wpc = WavpackOpenFileInput(fname, error, + OPEN_TAGS | OPEN_WVC | OPEN_NORMALIZE, + 23); if (wpc == nullptr) { FormatWarning(wavpack_domain, "failed to open WavPack file \"%s\": %s", -- cgit v1.2.3