From 4ee8da2e694d29efef5f6311d3bf8504d474918f Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Fri, 17 Oct 2008 21:57:09 +0200 Subject: Makefile.am: don't compile disabled decoder plugins Don't compile the sources of disabled decoder plugins at all, and don't attempt to register these. --- src/Makefile.am | 64 +++++++++++++++++++++++++++---------- src/decoder_list.c | 18 +++++++++++ src/inputPlugins/_flac_common.c | 7 ---- src/inputPlugins/_flac_common.h | 4 --- src/inputPlugins/_ogg_common.c | 8 +---- src/inputPlugins/_ogg_common.h | 5 --- src/inputPlugins/aac_plugin.c | 8 ----- src/inputPlugins/audiofile_plugin.c | 9 ------ src/inputPlugins/flac_plugin.c | 49 ++++++++-------------------- src/inputPlugins/mod_plugin.c | 9 ------ src/inputPlugins/mp3_plugin.c | 14 ++------ src/inputPlugins/mp4_plugin.c | 9 ------ src/inputPlugins/mpc_plugin.c | 9 ------ src/inputPlugins/oggflac_plugin.c | 9 ------ src/inputPlugins/oggvorbis_plugin.c | 11 ------- src/inputPlugins/wavpack_plugin.c | 9 ------ 16 files changed, 83 insertions(+), 159 deletions(-) (limited to 'src') diff --git a/src/Makefile.am b/src/Makefile.am index f36ea548..70222db0 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -15,21 +15,6 @@ mpd_audioOutputs = \ audioOutputs/audioOutput_mvp.c \ audioOutputs/audioOutput_jack.c -mpd_inputPlugins = \ - inputPlugins/_flac_common.c \ - inputPlugins/_ogg_common.c \ - inputPlugins/oggflac_plugin.c \ - inputPlugins/oggvorbis_plugin.c \ - inputPlugins/aac_plugin.c \ - inputPlugins/audiofile_plugin.c \ - inputPlugins/flac_plugin.c \ - inputPlugins/mod_plugin.c \ - inputPlugins/mp3_plugin.c \ - inputPlugins/mp4_plugin.c \ - inputPlugins/mpc_plugin.c \ - inputPlugins/wavpack_plugin.c - - mpd_headers = \ notify.h \ ack.h \ @@ -118,7 +103,6 @@ mpd_headers = \ mpd_SOURCES = \ $(mpd_headers) \ $(mpd_audioOutputs) \ - $(mpd_inputPlugins) \ notify.c \ audio.c \ audioOutput.c \ @@ -194,6 +178,54 @@ if HAVE_ID3TAG mpd_SOURCES += tag_id3.c endif + +# decoder plugins + +if HAVE_MAD +mpd_SOURCES += inputPlugins/mp3_plugin.c +endif + +if HAVE_MPCDEC +mpd_SOURCES += inputPlugins/mpc_plugin.c +endif + +if HAVE_WAVPACK +mpd_SOURCES += inputPlugins/wavpack_plugin.c +endif + +if HAVE_FAAD +mpd_SOURCES += inputPlugins/aac_plugin.c inputPlugins/mp4_plugin.c +endif + +if HAVE_OGG_COMMON +mpd_SOURCES += inputPlugins/_ogg_common.c +endif + +if HAVE_FLAC_COMMON +mpd_SOURCES += inputPlugins/_flac_common.c +endif + +if HAVE_OGGVORBIS +mpd_SOURCES += inputPlugins/oggvorbis_plugin.c +endif + +if HAVE_FLAC +mpd_SOURCES += inputPlugins/flac_plugin.c +endif + +if HAVE_OGGFLAC +mpd_SOURCES += inputPlugins/oggflac_plugin.c +endif + +if HAVE_AUDIOFILE +mpd_SOURCES += inputPlugins/audiofile_plugin.c +endif + +if HAVE_MIKMOD +mpd_SOURCES += inputPlugins/mod_plugin.c +endif + + if HAVE_ZEROCONF mpd_SOURCES += zeroconf.c endif diff --git a/src/decoder_list.c b/src/decoder_list.c index 6626f556..4aab73bd 100644 --- a/src/decoder_list.c +++ b/src/decoder_list.c @@ -151,16 +151,34 @@ void decoder_plugin_init_all(void) inputPlugin_list = makeList(NULL, 1); /* load plugins here */ +#ifdef HAVE_MAD decoder_plugin_load(&mp3Plugin); +#endif +#ifdef HAVE_OGGVORBIS decoder_plugin_load(&oggvorbisPlugin); +#endif +#ifdef HAVE_FLAC_COMMON decoder_plugin_load(&oggflacPlugin); +#endif +#ifdef HAVE_FLAC decoder_plugin_load(&flacPlugin); +#endif +#ifdef HAVE_AUDIOFILE decoder_plugin_load(&audiofilePlugin); +#endif +#ifdef HAVE_FAAD decoder_plugin_load(&mp4Plugin); decoder_plugin_load(&aacPlugin); +#endif +#ifdef HAVE_MPCDEC decoder_plugin_load(&mpcPlugin); +#endif +#ifdef HAVE_WAVPACK decoder_plugin_load(&wavpackPlugin); +#endif +#ifdef HAVE_MIKMOD decoder_plugin_load(&modPlugin); +#endif } void decoder_plugin_deinit_all(void) diff --git a/src/inputPlugins/_flac_common.c b/src/inputPlugins/_flac_common.c index f24e2053..db43e000 100644 --- a/src/inputPlugins/_flac_common.c +++ b/src/inputPlugins/_flac_common.c @@ -19,12 +19,7 @@ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -#include "../decoder_api.h" - -#if defined(HAVE_FLAC) || defined(HAVE_OGGFLAC) - #include "_flac_common.h" - #include "../log.h" #include @@ -326,5 +321,3 @@ flac_common_write(FlacData *data, const FLAC__Frame * frame, return FLAC__STREAM_DECODER_WRITE_STATUS_CONTINUE; } - -#endif /* HAVE_FLAC || HAVE_OGGFLAC */ diff --git a/src/inputPlugins/_flac_common.h b/src/inputPlugins/_flac_common.h index e0582414..45714b4b 100644 --- a/src/inputPlugins/_flac_common.h +++ b/src/inputPlugins/_flac_common.h @@ -24,8 +24,6 @@ #include "../decoder_api.h" -#if defined(HAVE_FLAC) || defined(HAVE_OGGFLAC) - #include #if !defined(FLAC_API_VERSION_CURRENT) || FLAC_API_VERSION_CURRENT <= 7 # include @@ -167,6 +165,4 @@ FLAC__StreamDecoderWriteStatus flac_common_write(FlacData *data, const FLAC__Frame * frame, const FLAC__int32 *const buf[]); -#endif /* HAVE_FLAC || HAVE_OGGFLAC */ - #endif /* _FLAC_COMMON_H */ diff --git a/src/inputPlugins/_ogg_common.c b/src/inputPlugins/_ogg_common.c index 535b9a9d..841b2ad3 100644 --- a/src/inputPlugins/_ogg_common.c +++ b/src/inputPlugins/_ogg_common.c @@ -19,12 +19,8 @@ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ -#include "_flac_common.h" #include "_ogg_common.h" - -#if defined(HAVE_OGGFLAC) || defined(HAVE_OGGVORBIS) || \ - (defined(FLAC_API_VERSION_CURRENT) && FLAC_API_VERSION_CURRENT > 7) - +#include "_flac_common.h" #include "../utils.h" ogg_stream_type ogg_stream_type_detect(InputStream * inStream) @@ -51,5 +47,3 @@ ogg_stream_type ogg_stream_type_detect(InputStream * inStream) } return VORBIS; } - -#endif /* defined(HAVE_OGGFLAC || defined(HAVE_OGGVORBIS) */ diff --git a/src/inputPlugins/_ogg_common.h b/src/inputPlugins/_ogg_common.h index 3382e286..7c9e7b63 100644 --- a/src/inputPlugins/_ogg_common.h +++ b/src/inputPlugins/_ogg_common.h @@ -24,13 +24,8 @@ #include "../decoder_api.h" -#if defined(HAVE_OGGFLAC) || defined(HAVE_OGGVORBIS) || \ - (defined(FLAC_API_VERSION_CURRENT) && FLAC_API_VERSION_CURRENT > 7) - typedef enum _ogg_stream_type { VORBIS, FLAC } ogg_stream_type; ogg_stream_type ogg_stream_type_detect(InputStream * inStream); -#endif /* defined(HAVE_OGGFLAC || defined(HAVE_OGGVORBIS) */ - #endif /* _OGG_COMMON_H */ diff --git a/src/inputPlugins/aac_plugin.c b/src/inputPlugins/aac_plugin.c index e9b2d747..7842bcc2 100644 --- a/src/inputPlugins/aac_plugin.c +++ b/src/inputPlugins/aac_plugin.c @@ -18,8 +18,6 @@ #include "../decoder_api.h" -#ifdef HAVE_FAAD - #define AAC_MAX_CHANNELS 6 #include "../utils.h" @@ -602,9 +600,3 @@ struct decoder_plugin aacPlugin = { .suffixes = aac_suffixes, .mime_types = aac_mimeTypes }; - -#else - -struct decoder_plugin aacPlugin; - -#endif /* HAVE_FAAD */ diff --git a/src/inputPlugins/audiofile_plugin.c b/src/inputPlugins/audiofile_plugin.c index 4c08074c..99846e85 100644 --- a/src/inputPlugins/audiofile_plugin.c +++ b/src/inputPlugins/audiofile_plugin.c @@ -19,9 +19,6 @@ */ #include "../decoder_api.h" - -#ifdef HAVE_AUDIOFILE - #include "../log.h" #include @@ -148,9 +145,3 @@ struct decoder_plugin audiofilePlugin = { .stream_types = INPUT_PLUGIN_STREAM_FILE, .suffixes = audiofileSuffixes, }; - -#else - -struct decoder_plugin audiofilePlugin; - -#endif /* HAVE_AUDIOFILE */ diff --git a/src/inputPlugins/flac_plugin.c b/src/inputPlugins/flac_plugin.c index 2f3ec88d..7b9fce27 100644 --- a/src/inputPlugins/flac_plugin.c +++ b/src/inputPlugins/flac_plugin.c @@ -17,9 +17,6 @@ */ #include "_flac_common.h" - -#ifdef HAVE_FLAC - #include "../utils.h" #include "../log.h" @@ -386,12 +383,8 @@ static int flac_decode(struct decoder * decoder, InputStream * inStream) return flac_decode_internal(decoder, inStream, 0); } -#if !defined(FLAC_API_VERSION_CURRENT) || FLAC_API_VERSION_CURRENT <= 7 -# define flac_plugin_init NULL -#else /* FLAC_API_VERSION_CURRENT >= 7 */ -/* some of this stuff is duplicated from oggflac_plugin.c */ -extern struct decoder_plugin oggflacPlugin; - +#if defined(FLAC_API_VERSION_CURRENT) && FLAC_API_VERSION_CURRENT > 7 && \ + !defined(HAVE_OGGFLAC) static struct tag *oggflac_tag_dup(char *file) { struct tag *ret = NULL; @@ -429,7 +422,8 @@ static int oggflac_decode(struct decoder *decoder, InputStream * inStream) static bool oggflac_try_decode(InputStream * inStream) { - return ogg_stream_type_detect(inStream) == FLAC; + return FLAC_API_SUPPORTS_OGG_FLAC && + ogg_stream_type_detect(inStream) == FLAC; } static const char *oggflac_suffixes[] = { "ogg", "oga", NULL }; @@ -438,25 +432,15 @@ static const char *oggflac_mime_types[] = { "audio/x-flac+ogg", "application/x-ogg", NULL }; -static int flac_plugin_init(void) -{ - if (!FLAC_API_SUPPORTS_OGG_FLAC) { - DEBUG("libFLAC does not support OggFLAC\n"); - return 1; - } - DEBUG("libFLAC supports OggFLAC, initializing OggFLAC support\n"); - assert(oggflacPlugin.name == NULL); - oggflacPlugin.name = "oggflac"; - oggflacPlugin.try_decode = oggflac_try_decode; - oggflacPlugin.stream_decode = oggflac_decode; - oggflacPlugin.tag_dup = oggflac_tag_dup; - oggflacPlugin.stream_types = INPUT_PLUGIN_STREAM_URL | - INPUT_PLUGIN_STREAM_FILE; - oggflacPlugin.suffixes = oggflac_suffixes; - oggflacPlugin.mime_types = oggflac_mime_types; - decoder_plugin_register(&oggflacPlugin); - return 1; -} +struct decoder_plugin oggflacPlugin = { + .name = "oggflac", + .try_decode = oggflac_try_decode, + .stream_decode = oggflac_decode, + .tag_dup = oggflac_tag_dup, + .stream_types = INPUT_PLUGIN_STREAM_URL | INPUT_PLUGIN_STREAM_FILE, + .suffixes = oggflac_suffixes, + .mime_types = oggflac_mime_types +}; #endif /* FLAC_API_VERSION_CURRENT >= 7 */ @@ -467,16 +451,9 @@ static const char *flac_mime_types[] = { "audio/x-flac", struct decoder_plugin flacPlugin = { .name = "flac", - .init = flac_plugin_init, .stream_decode = flac_decode, .tag_dup = flacTagDup, .stream_types = INPUT_PLUGIN_STREAM_URL | INPUT_PLUGIN_STREAM_FILE, .suffixes = flacSuffixes, .mime_types = flac_mime_types }; - -#else /* !HAVE_FLAC */ - -struct decoder_plugin flacPlugin; - -#endif /* HAVE_FLAC */ diff --git a/src/inputPlugins/mod_plugin.c b/src/inputPlugins/mod_plugin.c index 98bd67f0..5916a24a 100644 --- a/src/inputPlugins/mod_plugin.c +++ b/src/inputPlugins/mod_plugin.c @@ -17,9 +17,6 @@ */ #include "../decoder_api.h" - -#ifdef HAVE_MIKMOD - #include "../utils.h" #include "../log.h" @@ -279,9 +276,3 @@ struct decoder_plugin modPlugin = { .stream_types = INPUT_PLUGIN_STREAM_FILE, .suffixes = modSuffixes, }; - -#else - -struct decoder_plugin modPlugin; - -#endif /* HAVE_MIKMOD */ diff --git a/src/inputPlugins/mp3_plugin.c b/src/inputPlugins/mp3_plugin.c index 13e4a461..18eddf01 100644 --- a/src/inputPlugins/mp3_plugin.c +++ b/src/inputPlugins/mp3_plugin.c @@ -17,8 +17,9 @@ */ #include "../decoder_api.h" - -#ifdef HAVE_MAD +#include "../log.h" +#include "../utils.h" +#include "../conf.h" #include @@ -26,10 +27,6 @@ #include #endif -#include "../log.h" -#include "../utils.h" -#include "../conf.h" - #define FRAMES_CUSHION 2000 #define READ_BUFFER_SIZE 40960 @@ -1125,8 +1122,3 @@ struct decoder_plugin mp3Plugin = { .suffixes = mp3_suffixes, .mime_types = mp3_mimeTypes }; -#else - -struct decoder_plugin mp3Plugin; - -#endif /* HAVE_MAD */ diff --git a/src/inputPlugins/mp4_plugin.c b/src/inputPlugins/mp4_plugin.c index 0e152c3a..1a8268e4 100644 --- a/src/inputPlugins/mp4_plugin.c +++ b/src/inputPlugins/mp4_plugin.c @@ -17,9 +17,6 @@ */ #include "../decoder_api.h" - -#ifdef HAVE_FAAD - #include "../utils.h" #include "../log.h" @@ -424,9 +421,3 @@ struct decoder_plugin mp4Plugin = { .suffixes = mp4_suffixes, .mime_types = mp4_mimeTypes, }; - -#else - -struct decoder_plugin mp4Plugin; - -#endif /* HAVE_FAAD */ diff --git a/src/inputPlugins/mpc_plugin.c b/src/inputPlugins/mpc_plugin.c index f74dc8dd..fb1b0b56 100644 --- a/src/inputPlugins/mpc_plugin.c +++ b/src/inputPlugins/mpc_plugin.c @@ -17,9 +17,6 @@ */ #include "../decoder_api.h" - -#ifdef HAVE_MPCDEC - #include "../utils.h" #include "../log.h" @@ -309,9 +306,3 @@ struct decoder_plugin mpcPlugin = { .stream_types = INPUT_PLUGIN_STREAM_URL | INPUT_PLUGIN_STREAM_FILE, .suffixes = mpcSuffixes, }; - -#else - -struct decoder_plugin mpcPlugin; - -#endif /* HAVE_MPCDEC */ diff --git a/src/inputPlugins/oggflac_plugin.c b/src/inputPlugins/oggflac_plugin.c index 53f233e0..091b0098 100644 --- a/src/inputPlugins/oggflac_plugin.c +++ b/src/inputPlugins/oggflac_plugin.c @@ -20,9 +20,6 @@ */ #include "_flac_common.h" - -#ifdef HAVE_OGGFLAC - #include "_ogg_common.h" #include "../utils.h" @@ -356,9 +353,3 @@ struct decoder_plugin oggflacPlugin = { .suffixes = oggflac_Suffixes, .mime_types = oggflac_mime_types }; - -#else /* !HAVE_FLAC */ - -struct decoder_plugin oggflacPlugin; - -#endif /* HAVE_OGGFLAC */ diff --git a/src/inputPlugins/oggvorbis_plugin.c b/src/inputPlugins/oggvorbis_plugin.c index bf244860..0eecb783 100644 --- a/src/inputPlugins/oggvorbis_plugin.c +++ b/src/inputPlugins/oggvorbis_plugin.c @@ -18,12 +18,7 @@ /* TODO 'ogg' should probably be replaced with 'oggvorbis' in all instances */ -#include "../decoder_api.h" - -#ifdef HAVE_OGGVORBIS - #include "_ogg_common.h" - #include "../utils.h" #include "../log.h" @@ -390,9 +385,3 @@ struct decoder_plugin oggvorbisPlugin = { .suffixes = oggvorbis_Suffixes, .mime_types = oggvorbis_MimeTypes }; - -#else /* !HAVE_OGGVORBIS */ - -struct decoder_plugin oggvorbisPlugin; - -#endif /* HAVE_OGGVORBIS */ diff --git a/src/inputPlugins/wavpack_plugin.c b/src/inputPlugins/wavpack_plugin.c index 3e99980b..14b7e5f6 100644 --- a/src/inputPlugins/wavpack_plugin.c +++ b/src/inputPlugins/wavpack_plugin.c @@ -19,9 +19,6 @@ */ #include "../decoder_api.h" - -#ifdef HAVE_WAVPACK - #include "../utils.h" #include "../log.h" #include "../path.h" @@ -575,9 +572,3 @@ struct decoder_plugin wavpackPlugin = { .suffixes = wavpackSuffixes, .mime_types = wavpackMimeTypes }; - -#else /* !HAVE_WAVPACK */ - -struct decoder_plugin wavpackPlugin; - -#endif /* !HAVE_WAVPACK */ -- cgit v1.2.3