From 13e9f18403b98f8123b1b983c4680957d684e47b Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Mon, 21 Oct 2013 20:36:34 +0200 Subject: [PATCH] DecoderPlugin: move functions into the struct --- src/DecoderList.cxx | 8 +- src/DecoderPlugin.cxx | 13 ++- src/DecoderPlugin.hxx | 176 ++++++++++++++++++---------------------- src/DecoderThread.cxx | 4 +- src/TagFile.cxx | 9 +- src/UpdateContainer.cxx | 4 +- test/read_tags.cxx | 6 +- test/run_decoder.cxx | 5 +- 8 files changed, 98 insertions(+), 127 deletions(-) diff --git a/src/DecoderList.cxx b/src/DecoderList.cxx index a9a4b5432..8fa821f58 100644 --- a/src/DecoderList.cxx +++ b/src/DecoderList.cxx @@ -148,7 +148,7 @@ decoder_plugin_from_suffix(const char *suffix, decoder_plugins[i] != nullptr; ++i) { plugin = decoder_plugins[i]; if (decoder_plugins_enabled[i] && - decoder_plugin_supports_suffix(*plugin, suffix)) + plugin->SupportsSuffix(suffix)) return plugin; } @@ -168,7 +168,7 @@ decoder_plugin_from_mime_type(const char *mimeType, unsigned int next) for (; decoder_plugins[i] != nullptr; ++i) { const struct DecoderPlugin *plugin = decoder_plugins[i]; if (decoder_plugins_enabled[i] && - decoder_plugin_supports_mime_type(*plugin, mimeType)) { + plugin->SupportsMimeType(mimeType)) { ++i; return plugin; } @@ -226,7 +226,7 @@ void decoder_plugin_init_all(void) /* the plugin is disabled in mpd.conf */ continue; - if (decoder_plugin_init(plugin, *param)) + if (plugin.Init(*param)) decoder_plugins_enabled[i] = true; } } @@ -234,5 +234,5 @@ void decoder_plugin_init_all(void) void decoder_plugin_deinit_all(void) { decoder_plugins_for_each_enabled(plugin) - decoder_plugin_finish(*plugin); + plugin->Finish(); } diff --git a/src/DecoderPlugin.cxx b/src/DecoderPlugin.cxx index eb83d542d..77ed90882 100644 --- a/src/DecoderPlugin.cxx +++ b/src/DecoderPlugin.cxx @@ -24,22 +24,19 @@ #include bool -decoder_plugin_supports_suffix(const DecoderPlugin &plugin, - const char *suffix) +DecoderPlugin::SupportsSuffix(const char *suffix) const { assert(suffix != nullptr); - return plugin.suffixes != nullptr && - string_array_contains(plugin.suffixes, suffix); + return suffixes != nullptr && string_array_contains(suffixes, suffix); } bool -decoder_plugin_supports_mime_type(const DecoderPlugin &plugin, - const char *mime_type) +DecoderPlugin::SupportsMimeType(const char *mime_type) const { assert(mime_type != nullptr); - return plugin.mime_types != nullptr && - string_array_contains(plugin.mime_types, mime_type); + return mime_types != nullptr && + string_array_contains(mime_types, mime_type); } diff --git a/src/DecoderPlugin.hxx b/src/DecoderPlugin.hxx index aaf7e5152..7aa508c53 100644 --- a/src/DecoderPlugin.hxx +++ b/src/DecoderPlugin.hxx @@ -20,6 +20,8 @@ #ifndef MPD_DECODER_PLUGIN_HXX #define MPD_DECODER_PLUGIN_HXX +#include "Compiler.h" + struct config_param; struct input_stream; struct Tag; @@ -100,105 +102,81 @@ struct DecoderPlugin { /* last element in these arrays must always be a nullptr: */ const char *const*suffixes; const char *const*mime_types; + + /** + * Initialize a decoder plugin. + * + * @param param a configuration block for this plugin, or nullptr if none + * is configured + * @return true if the plugin was initialized successfully, false if + * the plugin is not available + */ + bool Init(const config_param ¶m) const { + return init != nullptr + ? init(param) + : true; + } + + /** + * Deinitialize a decoder plugin which was initialized successfully. + */ + void Finish() const { + if (finish != nullptr) + finish(); + } + + /** + * Decode a stream. + */ + void StreamDecode(decoder &decoder, input_stream &is) const { + stream_decode(&decoder, &is); + } + + /** + * Decode a file. + */ + void FileDecode(decoder &decoder, const char *path_fs) const { + file_decode(&decoder, path_fs); + } + + /** + * Read the tag of a file. + */ + bool ScanFile(const char *path_fs, + const tag_handler &handler, void *handler_ctx) const { + return scan_file != nullptr + ? scan_file(path_fs, &handler, handler_ctx) + : false; + } + + /** + * Read the tag of a stream. + */ + bool ScanStream(input_stream &is, + const tag_handler &handler, void *handler_ctx) const { + return scan_stream != nullptr + ? scan_stream(&is, &handler, handler_ctx) + : false; + } + + /** + * return "virtual" tracks in a container + */ + char *ContainerScan(const char *path, const unsigned int tnum) const { + return container_scan(path, tnum); + } + + /** + * Does the plugin announce the specified file name suffix? + */ + gcc_pure gcc_nonnull_all + bool SupportsSuffix(const char *suffix) const; + + /** + * Does the plugin announce the specified MIME type? + */ + gcc_pure gcc_nonnull_all + bool SupportsMimeType(const char *mime_type) const; }; -/** - * Initialize a decoder plugin. - * - * @param param a configuration block for this plugin, or nullptr if none - * is configured - * @return true if the plugin was initialized successfully, false if - * the plugin is not available - */ -static inline bool -decoder_plugin_init(const DecoderPlugin &plugin, - const config_param ¶m) -{ - return plugin.init != nullptr - ? plugin.init(param) - : true; -} - -/** - * Deinitialize a decoder plugin which was initialized successfully. - */ -static inline void -decoder_plugin_finish(const DecoderPlugin &plugin) -{ - if (plugin.finish != nullptr) - plugin.finish(); -} - -/** - * Decode a stream. - */ -static inline void -decoder_plugin_stream_decode(const DecoderPlugin &plugin, - struct decoder *decoder, struct input_stream *is) -{ - plugin.stream_decode(decoder, is); -} - -/** - * Decode a file. - */ -static inline void -decoder_plugin_file_decode(const DecoderPlugin &plugin, - struct decoder *decoder, const char *path_fs) -{ - plugin.file_decode(decoder, path_fs); -} - -/** - * Read the tag of a file. - */ -static inline bool -decoder_plugin_scan_file(const DecoderPlugin &plugin, - const char *path_fs, - const struct tag_handler *handler, void *handler_ctx) -{ - return plugin.scan_file != nullptr - ? plugin.scan_file(path_fs, handler, handler_ctx) - : false; -} - -/** - * Read the tag of a stream. - */ -static inline bool -decoder_plugin_scan_stream(const DecoderPlugin &plugin, - struct input_stream *is, - const struct tag_handler *handler, - void *handler_ctx) -{ - return plugin.scan_stream != nullptr - ? plugin.scan_stream(is, handler, handler_ctx) - : false; -} - -/** - * return "virtual" tracks in a container - */ -static inline char * -decoder_plugin_container_scan( const DecoderPlugin &plugin, - const char* pathname, - const unsigned int tnum) -{ - return plugin.container_scan(pathname, tnum); -} - -/** - * Does the plugin announce the specified file name suffix? - */ -bool -decoder_plugin_supports_suffix(const DecoderPlugin &plugin, - const char *suffix); - -/** - * Does the plugin announce the specified MIME type? - */ -bool -decoder_plugin_supports_mime_type(const DecoderPlugin &plugin, - const char *mime_type); - #endif diff --git a/src/DecoderThread.cxx b/src/DecoderThread.cxx index 83de64670..6fe1ff746 100644 --- a/src/DecoderThread.cxx +++ b/src/DecoderThread.cxx @@ -133,7 +133,7 @@ decoder_stream_decode(const DecoderPlugin &plugin, decoder->dc.Unlock(); - decoder_plugin_stream_decode(plugin, decoder, input_stream); + plugin.StreamDecode(*decoder, *input_stream); decoder->dc.Lock(); @@ -162,7 +162,7 @@ decoder_file_decode(const DecoderPlugin &plugin, decoder->dc.Unlock(); - decoder_plugin_file_decode(plugin, decoder, path); + plugin.FileDecode(*decoder, path); decoder->dc.Lock(); diff --git a/src/TagFile.cxx b/src/TagFile.cxx index 70e92e73f..c26bdecde 100644 --- a/src/TagFile.cxx +++ b/src/TagFile.cxx @@ -53,8 +53,8 @@ tag_file_scan(const char *path_fs, do { /* load file tag */ - if (decoder_plugin_scan_file(*plugin, path_fs, - handler, handler_ctx)) + if (plugin->ScanFile(path_fs, + *handler, handler_ctx)) break; /* fall back to stream tag */ @@ -67,9 +67,8 @@ tag_file_scan(const char *path_fs, /* now try the stream_tag() method */ if (is != nullptr) { - if (decoder_plugin_scan_stream(*plugin, is, - handler, - handler_ctx)) + if (plugin->ScanStream(*is, + *handler, handler_ctx)) break; is->LockSeek(0, SEEK_SET, IgnoreError()); diff --git a/src/UpdateContainer.cxx b/src/UpdateContainer.cxx index 94a004f12..3f658cbcc 100644 --- a/src/UpdateContainer.cxx +++ b/src/UpdateContainer.cxx @@ -98,8 +98,8 @@ update_container_file(Directory &directory, const auto child_path_fs = map_directory_child_fs(*contdir, vtrack); - decoder_plugin_scan_file(plugin, child_path_fs.c_str(), - &add_tag_handler, &tag_builder); + plugin.ScanFile(child_path_fs.c_str(), + add_tag_handler, &tag_builder); if (tag_builder.IsDefined()) song->tag = tag_builder.Commit(); diff --git a/test/read_tags.cxx b/test/read_tags.cxx index c7756524f..a3d48be47 100644 --- a/test/read_tags.cxx +++ b/test/read_tags.cxx @@ -181,8 +181,7 @@ int main(int argc, char **argv) return 1; } - bool success = decoder_plugin_scan_file(*plugin, path, - &print_handler, NULL); + bool success = plugin->ScanFile(path, print_handler, nullptr); if (!success && plugin->scan_stream != NULL) { Mutex mutex; Cond cond; @@ -209,8 +208,7 @@ int main(int argc, char **argv) mutex.unlock(); - success = decoder_plugin_scan_stream(*plugin, is, - &print_handler, NULL); + success = plugin->ScanStream(*is, print_handler, nullptr); is->Close(); } diff --git a/test/run_decoder.cxx b/test/run_decoder.cxx index 08f6f66b5..cd61d3beb 100644 --- a/test/run_decoder.cxx +++ b/test/run_decoder.cxx @@ -184,8 +184,7 @@ int main(int argc, char **argv) decoder.initialized = false; if (decoder.plugin->file_decode != NULL) { - decoder_plugin_file_decode(*decoder.plugin, &decoder, - decoder.uri); + decoder.plugin->FileDecode(decoder, decoder.uri); } else if (decoder.plugin->stream_decode != NULL) { Mutex mutex; Cond cond; @@ -201,7 +200,7 @@ int main(int argc, char **argv) return 1; } - decoder_plugin_stream_decode(*decoder.plugin, &decoder, is); + decoder.plugin->StreamDecode(decoder, *is); is->Close(); } else {