aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2013-07-28 12:54:59 +0200
committerMax Kellermann <max@duempel.org>2013-07-28 13:03:04 +0200
commit1688b6dda9c510a5737ae3eb192b56166e497fc5 (patch)
tree1cfecc376dc36ce91eee8965cb9562545fdc59ac
parent33aedc887acc4c52e140ba93df55f1825a2ebb10 (diff)
downloadmpd-1688b6dda9c510a5737ae3eb192b56166e497fc5.tar.gz
mpd-1688b6dda9c510a5737ae3eb192b56166e497fc5.tar.xz
mpd-1688b6dda9c510a5737ae3eb192b56166e497fc5.zip
decoder/mikmod: convert to C++
-rw-r--r--Makefile.am4
-rw-r--r--src/DecoderList.cxx2
-rw-r--r--src/decoder/MikmodDecoderPlugin.cxx (renamed from src/decoder/mikmod_decoder_plugin.c)41
-rw-r--r--src/decoder/MikmodDecoderPlugin.hxx25
4 files changed, 52 insertions, 20 deletions
diff --git a/Makefile.am b/Makefile.am
index a5b782037..f87e42654 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -574,7 +574,9 @@ libdecoder_plugins_a_SOURCES += \
endif
if ENABLE_MIKMOD_DECODER
-libdecoder_plugins_a_SOURCES += src/decoder/mikmod_decoder_plugin.c
+libdecoder_plugins_a_SOURCES += \
+ src/decoder/MikmodDecoderPlugin.cxx \
+ src/decoder/MikmodDecoderPlugin.hxx
endif
if HAVE_MODPLUG
diff --git a/src/DecoderList.cxx b/src/DecoderList.cxx
index f6b029f7a..67fbed5f6 100644
--- a/src/DecoderList.cxx
+++ b/src/DecoderList.cxx
@@ -38,6 +38,7 @@
#include "decoder/SndfileDecoderPlugin.hxx"
#include "decoder/Mpg123DecoderPlugin.hxx"
#include "decoder/WildmidiDecoderPlugin.hxx"
+#include "decoder/MikmodDecoderPlugin.hxx"
#include <glib.h>
@@ -45,7 +46,6 @@
extern const struct decoder_plugin mpcdec_decoder_plugin;
extern const struct decoder_plugin modplug_decoder_plugin;
-extern const struct decoder_plugin mikmod_decoder_plugin;
extern const struct decoder_plugin sidplay_decoder_plugin;
extern const struct decoder_plugin fluidsynth_decoder_plugin;
diff --git a/src/decoder/mikmod_decoder_plugin.c b/src/decoder/MikmodDecoderPlugin.cxx
index 5681a7a57..94dff0856 100644
--- a/src/decoder/mikmod_decoder_plugin.c
+++ b/src/decoder/MikmodDecoderPlugin.cxx
@@ -1,5 +1,5 @@
/*
- * Copyright (C) 2003-2011 The Music Player Daemon Project
+ * Copyright (C) 2003-2013 The Music Player Daemon Project
* http://www.musicpd.org
*
* This program is free software; you can redistribute it and/or modify
@@ -18,6 +18,7 @@
*/
#include "config.h"
+#include "MikmodDecoderPlugin.hxx"
#include "decoder_api.h"
#include "mpd_error.h"
#include "tag_handler.h"
@@ -31,7 +32,7 @@
/* this is largely copied from alsaplayer */
-#define MIKMOD_FRAME_SIZE 4096
+static constexpr size_t MIKMOD_FRAME_SIZE = 4096;
static BOOL
mikmod_mpd_init(void)
@@ -64,7 +65,7 @@ static char drv_alias[] = PACKAGE;
#endif
static MDRIVER drv_mpd = {
- NULL,
+ nullptr,
drv_name,
drv_version,
0,
@@ -72,9 +73,9 @@ static MDRIVER drv_mpd = {
#if (LIBMIKMOD_VERSION > 0x030106)
drv_alias,
#if (LIBMIKMOD_VERSION >= 0x030200)
- NULL, /* CmdLineHelp */
+ nullptr, /* CmdLineHelp */
#endif
- NULL, /* CommandLine */
+ nullptr, /* CommandLine */
#endif
mikmod_mpd_is_present,
VC_SampleLoad,
@@ -83,12 +84,12 @@ static MDRIVER drv_mpd = {
VC_SampleLength,
mikmod_mpd_init,
mikmod_mpd_exit,
- NULL,
+ nullptr,
VC_SetNumVoices,
VC_PlayStart,
VC_PlayStop,
mikmod_mpd_update,
- NULL,
+ nullptr,
VC_VoiceSetVolume,
VC_VoiceGetVolume,
VC_VoiceSetFrequency,
@@ -155,7 +156,7 @@ mikmod_decoder_file_decode(struct decoder *decoder, const char *path_fs)
handle = Player_Load(path2, 128, 0);
g_free(path2);
- if (handle == NULL) {
+ if (handle == nullptr) {
g_warning("failed to open mod: %s", path_fs);
return;
}
@@ -171,7 +172,7 @@ mikmod_decoder_file_decode(struct decoder *decoder, const char *path_fs)
Player_Start(handle);
while (cmd == DECODE_COMMAND_NONE && Player_Active()) {
ret = VC_WriteBytes(buffer, sizeof(buffer));
- cmd = decoder_data(decoder, NULL, buffer, ret, 0);
+ cmd = decoder_data(decoder, nullptr, buffer, ret, 0);
}
Player_Stop();
@@ -185,7 +186,7 @@ mikmod_decoder_scan_file(const char *path_fs,
char *path2 = g_strdup(path_fs);
MODULE *handle = Player_Load(path2, 128, 0);
- if (handle == NULL) {
+ if (handle == nullptr) {
g_free(path2);
g_debug("Failed to open file: %s", path_fs);
return false;
@@ -197,7 +198,7 @@ mikmod_decoder_scan_file(const char *path_fs,
char *title = Player_LoadTitle(path2);
g_free(path2);
- if (title != NULL) {
+ if (title != nullptr) {
tag_handler_invoke_tag(handler, handler_ctx,
TAG_TITLE, title);
free(title);
@@ -222,14 +223,18 @@ static const char *const mikmod_decoder_suffixes[] = {
"ult",
"uni",
"xm",
- NULL
+ nullptr
};
const struct decoder_plugin mikmod_decoder_plugin = {
- .name = "mikmod",
- .init = mikmod_decoder_init,
- .finish = mikmod_decoder_finish,
- .file_decode = mikmod_decoder_file_decode,
- .scan_file = mikmod_decoder_scan_file,
- .suffixes = mikmod_decoder_suffixes,
+ "mikmod",
+ mikmod_decoder_init,
+ mikmod_decoder_finish,
+ nullptr,
+ mikmod_decoder_file_decode,
+ mikmod_decoder_scan_file,
+ nullptr,
+ nullptr,
+ mikmod_decoder_suffixes,
+ nullptr,
};
diff --git a/src/decoder/MikmodDecoderPlugin.hxx b/src/decoder/MikmodDecoderPlugin.hxx
new file mode 100644
index 000000000..dd3b1389e
--- /dev/null
+++ b/src/decoder/MikmodDecoderPlugin.hxx
@@ -0,0 +1,25 @@
+/*
+ * Copyright (C) 2003-2013 The Music Player Daemon Project
+ * http://www.musicpd.org
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
+
+#ifndef MPD_DECODER_MIKMOD_HXX
+#define MPD_DECODER_MIKMOD_HXX
+
+extern const struct decoder_plugin mikmod_decoder_plugin;
+
+#endif