This releases several include file dependencies.  As a side effect,
"CHUNK_SIZE" isn't defined by decoder_api.h anymore, so we have to
define it directly in the plugins which need it.  It just isn't worth
it to add it to the decoder plugin API.
---

 src/decoder_api.c                   |    1 +
 src/inputPlugins/audiofile_plugin.c |    3 +++
 src/inputPlugins/wavpack_plugin.c   |    3 +++
 src/outputBuffer.c                  |    3 ++-
 src/outputBuffer.h                  |    6 ++++--
 src/playerData.c                    |    2 +-
 src/playerData.h                    |    2 --
 7 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/src/decoder_api.c b/src/decoder_api.c
index 6d8ce08..f6119aa 100644
--- a/src/decoder_api.c
+++ b/src/decoder_api.c
@@ -25,6 +25,7 @@
 #include "utils.h"
 #include "normalize.h"
 #include "playerData.h"
+#include "outputBuffer.h"
 #include "gcc.h"
 
 void decoder_plugin_register(struct decoder_plugin *plugin)
diff --git a/src/inputPlugins/audiofile_plugin.c 
b/src/inputPlugins/audiofile_plugin.c
index 2fbe580..c304677 100644
--- a/src/inputPlugins/audiofile_plugin.c
+++ b/src/inputPlugins/audiofile_plugin.c
@@ -26,6 +26,9 @@
 
 #include <audiofile.h>
 
+/* pick 1020 since its devisible for 8,16,24, and 32-bit audio */
+#define CHUNK_SIZE             1020
+
 static int getAudiofileTotalTime(char *file)
 {
        int total_time;
diff --git a/src/inputPlugins/wavpack_plugin.c 
b/src/inputPlugins/wavpack_plugin.c
index 7a37e69..2253feb 100644
--- a/src/inputPlugins/wavpack_plugin.c
+++ b/src/inputPlugins/wavpack_plugin.c
@@ -28,6 +28,9 @@
 
 #include <wavpack/wavpack.h>
 
+/* pick 1020 since its devisible for 8,16,24, and 32-bit audio */
+#define CHUNK_SIZE             1020
+
 #define ERRORLEN 80
 
 static struct {
diff --git a/src/outputBuffer.c b/src/outputBuffer.c
index e500e68..0efc5d1 100644
--- a/src/outputBuffer.c
+++ b/src/outputBuffer.c
@@ -17,10 +17,11 @@
  */
 
 #include "outputBuffer.h"
-#include "playerData.h"
 
 #include "utils.h"
 
+struct output_buffer ob;
+
 void ob_init(unsigned int size, Notify *notify)
 {
        assert(size > 0);
diff --git a/src/outputBuffer.h b/src/outputBuffer.h
index 624b6db..512334d 100644
--- a/src/outputBuffer.h
+++ b/src/outputBuffer.h
@@ -36,7 +36,7 @@ typedef struct _OutputBufferChunk {
  * A ring set of buffers where the decoder appends data after the end,
  * and the player consumes data from the beginning.
  */
-typedef struct _OutputBuffer {
+struct output_buffer {
        ob_chunk *chunks;
 
        unsigned int size;
@@ -54,7 +54,9 @@ typedef struct _OutputBuffer {
        AudioFormat audioFormat;
 
        Notify *notify;
-} OutputBuffer;
+};
+
+extern struct output_buffer ob;
 
 void ob_init(unsigned int size, Notify *notify);
 
diff --git a/src/playerData.c b/src/playerData.c
index 81c6f7f..d72f2fe 100644
--- a/src/playerData.c
+++ b/src/playerData.c
@@ -18,6 +18,7 @@
 
 #include "playerData.h"
 #include "decode.h"
+#include "outputBuffer.h"
 #include "conf.h"
 #include "log.h"
 #include "utils.h"
@@ -27,7 +28,6 @@
 
 unsigned int buffered_before_play;
 PlayerControl pc;
-OutputBuffer ob;
 
 void initPlayerData(void)
 {
diff --git a/src/playerData.h b/src/playerData.h
index 81b80cf..a469ffa 100644
--- a/src/playerData.h
+++ b/src/playerData.h
@@ -20,11 +20,9 @@
 #define PLAYER_DATA_H
 
 #include "player.h"
-#include "outputBuffer.h"
 
 extern unsigned int buffered_before_play;
 extern PlayerControl pc;
-extern OutputBuffer ob;
 
 void initPlayerData(void);
 


-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Musicpd-dev-team mailing list
Musicpd-dev-team@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/musicpd-dev-team

Reply via email to