aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorEric Wong <normalperson@yhbt.net>2008-09-29 13:29:33 +0200
committerMax Kellermann <max@duempel.org>2008-09-29 13:29:33 +0200
commit0352766dca3da5266e4714124fea119be82c4188 (patch)
treea494e59ecd91f9a72afff65dcfc12bc508f4eb9f /src
parenta4019f7d2183b3ab6c2bbef68c7a5259326bc973 (diff)
Switch to C99 types (retaining compat with old compilers)
Seeing the "mpd_" prefix _everywhere_ is mind-numbing as the mind needs to retrain itself to skip over the first 4 tokens of a type to get to its meaning. So avoid having extra characters on my terminal to make it easier to follow code at 2:30 am in the morning. Please report any new issues you may come across on Free toolchains. I realize how difficult it can be to build/maintain cross-compiling toolchains and I have no intention of forcing people to upgrade their toolchains to build mpd. Tested with gcc 2.95.4 and and gcc 4.3.1 on x86-32.
Diffstat (limited to 'src')
-rw-r--r--src/audio.c6
-rw-r--r--src/audioOutputs/audioOutput_alsa.c2
-rw-r--r--src/audioOutputs/audioOutput_oss.c4
-rw-r--r--src/charConv.c6
-rw-r--r--src/dbUtils.c2
-rw-r--r--src/inputPlugins/aac_plugin.c4
-rw-r--r--src/inputPlugins/mp3_plugin.c10
-rw-r--r--src/inputPlugins/mp4_plugin.c2
-rw-r--r--src/locate.c4
-rw-r--r--src/locate.h2
-rw-r--r--src/metadata_pipe.c151
-rw-r--r--src/os_compat.h9
-rw-r--r--src/outputBuffer_accessors.h88
-rw-r--r--src/outputBuffer_ob_send.h133
-rw-r--r--src/pcm_utils.c64
-rw-r--r--src/pcm_utils.h6
-rw-r--r--src/replayGain.c10
-rw-r--r--src/tag.c2
-rw-r--r--src/tag.h2
-rw-r--r--src/tag_internal.h2
20 files changed, 437 insertions, 72 deletions
diff --git a/src/audio.c b/src/audio.c
index e79d0e54..bbc7fd0e 100644
--- a/src/audio.c
+++ b/src/audio.c
@@ -44,7 +44,7 @@ enum ad_state {
and enable in playAudio() routine */
static enum ad_state *audioDeviceStates;
-static mpd_uint8 audioOpened;
+static uint8_t audioOpened;
static struct {
struct audio_format format;
@@ -150,7 +150,7 @@ int parseAudioConfig(struct audio_format *audioFormat, char *conf)
return -1;
}
- audioFormat->bits = (mpd_sint8)strtol(test + 1, &test, 10);
+ audioFormat->bits = (int8_t)strtol(test + 1, &test, 10);
if (*test != ':') {
ERROR("error parsing audio output format: %s\n", conf);
@@ -166,7 +166,7 @@ int parseAudioConfig(struct audio_format *audioFormat, char *conf)
return -1;
}
- audioFormat->channels = (mpd_sint8)strtol(test + 1, &test, 10);
+ audioFormat->channels = (int8_t)strtol(test + 1, &test, 10);
if (*test != '\0') {
ERROR("error parsing audio output format: %s\n", conf);
diff --git a/src/audioOutputs/audioOutput_alsa.c b/src/audioOutputs/audioOutput_alsa.c
index 98a7155d..e2146696 100644
--- a/src/audioOutputs/audioOutput_alsa.c
+++ b/src/audioOutputs/audioOutput_alsa.c
@@ -214,7 +214,7 @@ configure_hw:
snd_strerror(-err));
goto fail;
}
- audioFormat->channels = (mpd_sint8)channels;
+ audioFormat->channels = (int8_t)channels;
err = snd_pcm_hw_params_set_rate_near(ad->pcmHandle, hwparams,
&sampleRate, NULL);
diff --git a/src/audioOutputs/audioOutput_oss.c b/src/audioOutputs/audioOutput_oss.c
index f5195ab4..ae483877 100644
--- a/src/audioOutputs/audioOutput_oss.c
+++ b/src/audioOutputs/audioOutput_oss.c
@@ -482,9 +482,9 @@ static int oss_openDevice(void *data,
int ret;
OssData *od = data;
- od->channels = (mpd_sint8)audioFormat->channels;
+ od->channels = (int8_t)audioFormat->channels;
od->sampleRate = audioFormat->sampleRate;
- od->bits = (mpd_sint8)audioFormat->bits;
+ od->bits = (int8_t)audioFormat->bits;
if ((ret = oss_open(od)) < 0)
return ret;
diff --git a/src/charConv.c b/src/charConv.c
index c205e93e..7d25fbff 100644
--- a/src/charConv.c
+++ b/src/charConv.c
@@ -32,13 +32,13 @@ static iconv_t char_conv_iconv;
static char *char_conv_to;
static char *char_conv_from;
-static mpd_sint8 char_conv_same;
-static mpd_sint8 char_conv_use_iconv;
+static int8_t char_conv_same;
+static int8_t char_conv_use_iconv;
/* 1 is to use latin1ToUtf8
0 is not to use latin1/utf8 converter
-1 is to use utf8ToLatin1*/
-static mpd_sint8 char_conv_latin1ToUtf8;
+static int8_t char_conv_latin1ToUtf8;
#define BUFFER_SIZE MPD_PATH_MAX
diff --git a/src/dbUtils.c b/src/dbUtils.c
index 98943d00..ac324062 100644
--- a/src/dbUtils.c
+++ b/src/dbUtils.c
@@ -30,7 +30,7 @@
#include "storedPlaylist.h"
typedef struct _ListCommandItem {
- mpd_sint8 tagType;
+ int8_t tagType;
int numConditionals;
LocateTagItem *conditionals;
} ListCommandItem;
diff --git a/src/inputPlugins/aac_plugin.c b/src/inputPlugins/aac_plugin.c
index dc93b26b..43ff6c1d 100644
--- a/src/inputPlugins/aac_plugin.c
+++ b/src/inputPlugins/aac_plugin.c
@@ -316,7 +316,7 @@ static int aac_stream_decode(struct decoder * mpd_decoder,
unsigned int sampleCount;
char *sampleBuffer;
size_t sampleBufferLen;
- mpd_uint16 bitRate = 0;
+ uint16_t bitRate = 0;
AacBuffer b;
int initialized = 0;
@@ -453,7 +453,7 @@ static int aac_decode(struct decoder * mpd_decoder, char *path)
/*float * seekTable;
long seekTableEnd = -1;
int seekPositionFound = 0; */
- mpd_uint16 bitRate = 0;
+ uint16_t bitRate = 0;
AacBuffer b;
InputStream inStream;
int initialized = 0;
diff --git a/src/inputPlugins/mp3_plugin.c b/src/inputPlugins/mp3_plugin.c
index 3e57a370..cbf76a3c 100644
--- a/src/inputPlugins/mp3_plugin.c
+++ b/src/inputPlugins/mp3_plugin.c
@@ -65,7 +65,7 @@ static unsigned long prng(unsigned long state)
return (state * 0x0019660dL + 0x3c6ef35fL) & 0xffffffffL;
}
-static mpd_sint16 audio_linear_dither(unsigned int bits, mad_fixed_t sample,
+static int16_t audio_linear_dither(unsigned int bits, mad_fixed_t sample,
struct audio_dither *dither)
{
unsigned int scalebits;
@@ -107,15 +107,15 @@ static mpd_sint16 audio_linear_dither(unsigned int bits, mad_fixed_t sample,
dither->error[0] = sample - output;
- return (mpd_sint16)(output >> scalebits);
+ return (int16_t)(output >> scalebits);
}
-static unsigned dither_buffer(mpd_sint16 *dest0, const struct mad_synth *synth,
+static unsigned dither_buffer(int16_t *dest0, const struct mad_synth *synth,
struct audio_dither *dither,
unsigned int start, unsigned int end,
unsigned int num_channels)
{
- mpd_sint16 *dest = dest0;
+ int16_t *dest = dest0;
unsigned int i;
for (i = start; i < end; ++i) {
@@ -153,7 +153,7 @@ typedef struct _mp3DecodeData {
struct mad_synth synth;
mad_timer_t timer;
unsigned char readBuffer[READ_BUFFER_SIZE];
- mpd_sint16 outputBuffer[MP3_DATA_OUTPUT_BUFFER_SIZE];
+ int16_t outputBuffer[MP3_DATA_OUTPUT_BUFFER_SIZE];
float totalTime;
float elapsedTime;
enum muteframe muteFrame;
diff --git a/src/inputPlugins/mp4_plugin.c b/src/inputPlugins/mp4_plugin.c
index 42e20599..6a999e4e 100644
--- a/src/inputPlugins/mp4_plugin.c
+++ b/src/inputPlugins/mp4_plugin.c
@@ -104,7 +104,7 @@ static int mp4_decode(struct decoder * mpd_decoder, InputStream * inStream)
long seekTableEnd = -1;
int seekPositionFound = 0;
long offset;
- mpd_uint16 bitRate = 0;
+ uint16_t bitRate = 0;
int seeking = 0;
double seek_where = 0;
int initialized = 0;
diff --git a/src/locate.c b/src/locate.c
index b9e1aa81..1ac0fbf9 100644
--- a/src/locate.c
+++ b/src/locate.c
@@ -127,7 +127,7 @@ static int strstrSearchTag(Song * song, enum tag_type type, char *str)
int i;
char *duplicate;
int ret = 0;
- mpd_sint8 visitedTypes[TAG_NUM_OF_ITEM_TYPES] = { 0 };
+ int8_t visitedTypes[TAG_NUM_OF_ITEM_TYPES] = { 0 };
if (type == LOCATE_TAG_FILE_TYPE || type == LOCATE_TAG_ANY_TYPE) {
char path_max_tmp[MPD_PATH_MAX];
@@ -184,7 +184,7 @@ int strstrSearchTags(Song * song, int numItems, LocateTagItem * items)
static int tagItemFoundAndMatches(Song * song, enum tag_type type, char *str)
{
int i;
- mpd_sint8 visitedTypes[TAG_NUM_OF_ITEM_TYPES] = { 0 };
+ int8_t visitedTypes[TAG_NUM_OF_ITEM_TYPES] = { 0 };
if (type == LOCATE_TAG_FILE_TYPE || type == LOCATE_TAG_ANY_TYPE) {
char path_max_tmp[MPD_PATH_MAX];
diff --git a/src/locate.h b/src/locate.h
index 325ae384..7a817828 100644
--- a/src/locate.h
+++ b/src/locate.h
@@ -26,7 +26,7 @@
/* struct used for search, find, list queries */
typedef struct _LocateTagItem {
- mpd_sint8 tagType;
+ int8_t tagType;
/* what we are looking for */
char *needle;
} LocateTagItem;
diff --git a/src/metadata_pipe.c b/src/metadata_pipe.c
new file mode 100644
index 00000000..83c36ce1
--- /dev/null
+++ b/src/metadata_pipe.c
@@ -0,0 +1,151 @@
+/* the Music Player Daemon (MPD)
+ * Copyright (C) 2003-2007 by Warren Dukes (warren.dukes@gmail.com)
+ * This project's homepage is: 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., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+#include "metadata_pipe.h"
+#include "ringbuf.h"
+#include "decode.h"
+#include "os_compat.h"
+#include "log.h"
+#include "outputBuffer.h"
+#include "gcc.h"
+
+static struct ringbuf *mp;
+
+/* Each one of these is a packet inside the metadata pipe */
+struct tag_container {
+ float metadata_time;
+ uint8_t seq; /* ob.seq_decoder at time of metadata_pipe_send() */
+ struct mpd_tag *tag; /* our payload */
+};
+
+/*
+ * We have two readers even though ringbuf was designed for one (locklessly),
+ * so we will use a lock to allow readers to safely read. Writing is only
+ * done from one thread, so it will never block or clobber.
+ */
+static pthread_mutex_t read_lock = PTHREAD_MUTEX_INITIALIZER;
+static struct mpd_tag *current_tag; /* requires read_lock for both r/w access */
+
+static void metadata_pipe_finish(void)
+{
+ ringbuf_free(mp);
+ if (current_tag)
+ tag_free(current_tag);
+}
+
+void init_metadata_pipe(void)
+{
+ mp = ringbuf_create(sizeof(struct tag_container) * 16);
+ atexit(metadata_pipe_finish);
+}
+
+void metadata_pipe_send(struct mpd_tag *tag, float metadata_time)
+{
+ struct tag_container tc;
+ size_t written;
+
+ assert(pthread_equal(pthread_self(), dc.thread));
+
+ if (mpd_unlikely(ringbuf_write_space(mp)
+ < sizeof(struct tag_container))) {
+ DEBUG("metadata_pipe: insufficient buffer space, dropping\n");
+ tag_free(tag);
+ return;
+ }
+
+ tc.tag = tag;
+ tc.metadata_time = metadata_time;
+ tc.seq = ob_get_decoder_sequence();
+ written = ringbuf_write(mp, &tc, sizeof(struct tag_container));
+ assert(written == sizeof(struct tag_container));
+}
+
+struct mpd_tag * metadata_pipe_recv(void)
+{
+ struct tag_container tc;
+ size_t r;
+ static const size_t uint8_t_max = 255; /* XXX CLEANUP */
+ uint8_t expect_seq = ob_get_player_sequence();
+ unsigned long current_time = ob_get_elapsed_time();
+ struct mpd_tag *tag = NULL;
+
+ if (pthread_mutex_trylock(&read_lock) == EBUSY)
+ return NULL;
+retry:
+ if (!(r = ringbuf_peek(mp, &tc, sizeof(struct tag_container))))
+ goto out;
+
+ assert(r == sizeof(struct tag_container));
+ assert(tc.tag);
+ if (expect_seq == tc.seq) {
+ if (current_time < tc.metadata_time)
+ goto out; /* not ready for tag yet */
+ if (tag_equal(tc.tag, current_tag)) {
+ tag_free(tc.tag);
+ ringbuf_read_advance(mp, sizeof(struct tag_container));
+ goto out; /* nothing changed, don't bother */
+ }
+ tag = tag_dup(tc.tag);
+ if (current_tag)
+ tag_free(current_tag);
+ current_tag = tc.tag;
+ ringbuf_read_advance(mp, sizeof(struct tag_container));
+ } else if (expect_seq > tc.seq ||
+ (!expect_seq && tc.seq == uint8_t_max)) {
+ DEBUG("metadata_pipe: reader is ahead of writer\n");
+ tag_free(tc.tag);
+ ringbuf_read_advance(mp, sizeof(struct tag_container));
+ goto retry; /* read and skip packets */
+ } /* else not ready for tag yet */
+out:
+ pthread_mutex_unlock(&read_lock);
+ return tag;
+}
+
+struct mpd_tag *metadata_pipe_current(void)
+{
+ struct mpd_tag *tag;
+
+ assert(! pthread_equal(pthread_self(), dc.thread));
+ if (pthread_mutex_trylock(&read_lock) == EBUSY)
+ return NULL;
+ tag = current_tag ? tag_dup(current_tag) : NULL;
+ pthread_mutex_unlock(&read_lock);
+
+ return tag;
+}
+
+void metadata_pipe_clear(void)
+{
+ struct tag_container tc;
+ size_t r;
+
+ pthread_mutex_lock(&read_lock);
+
+ while ((r = ringbuf_read(mp, &tc, sizeof(struct tag_container)))) {
+ assert(r == sizeof(struct tag_container));
+ tag_free(tc.tag);
+ }
+
+ if (current_tag) {
+ tag_free(current_tag);
+ current_tag = NULL;
+ }
+
+ pthread_mutex_unlock(&read_lock);
+}
diff --git a/src/os_compat.h b/src/os_compat.h
index 7cd47cbf..c3aede51 100644
--- a/src/os_compat.h
+++ b/src/os_compat.h
@@ -31,14 +31,7 @@
* 2) optional features in core (libsamplerate, avahi, ...)
*/
-#if defined(HAVE_STDINT_H)
-#include <stdint.h>
-#elif defined(HAVE_INTTYPES_H)
-#include <inttypes.h>
-#elif defined(HAVE_SYS_INTTYPES_H)
-#include <sys/inttypes.h>
-#endif
-#include <sys/types.h>
+#include "mpd_types.h"
#define _XOPEN_SOURCE 600 /* for posix_fadvise, won't hurt if not available */
#include <fcntl.h>
diff --git a/src/outputBuffer_accessors.h b/src/outputBuffer_accessors.h
new file mode 100644
index 00000000..8d59429d
--- /dev/null
+++ b/src/outputBuffer_accessors.h
@@ -0,0 +1,88 @@
+/* the Music Player Daemon (MPD)
+ * Copyright (C) 2003-2007 by Warren Dukes (warren.dukes@gmail.com)
+ * This project's homepage is: 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., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+/*
+ * Some of the most boring code in the world that I didn't want cluttering
+ * up outputBuffer.c
+ */
+
+unsigned long ob_get_elapsed_time(void)
+{
+ return (unsigned long)(ob.elapsed_time + 0.5);
+}
+
+unsigned long ob_get_total_time(void)
+{
+ return (unsigned long)(ob.total_time + 0.5);
+}
+
+unsigned int ob_get_bit_rate(void)
+{
+ return (unsigned int)ob.bit_rate;
+}
+
+unsigned int ob_get_channels(void)
+{
+ return (unsigned int)ob.audio_format.channels;
+}
+
+unsigned int ob_get_sample_rate(void)
+{
+ return (unsigned int)ob.audio_format.sampleRate;
+}
+
+unsigned int ob_get_bits(void)
+{
+ return (unsigned int)ob.audio_format.bits;
+}
+
+void ob_set_sw_volume(int volume)
+{
+ ob.sw_vol = (volume > 1000) ? 1000 : (volume < 0 ? 0 : volume);
+}
+
+void ob_set_xfade(float xfade_sec)
+{
+ ob.xfade_time = (xfade_sec < 0) ? 0 : xfade_sec;
+}
+
+float ob_get_xfade(void)
+{
+ return ob.xfade_time;
+}
+
+enum ob_state ob_get_state(void)
+{
+ return ob.state;
+}
+
+AudioFormat *ob_audio_format(void)
+{
+ return &ob.audio_format;
+}
+
+uint8_t ob_get_decoder_sequence(void)
+{
+ return ob.seq_decoder;
+}
+
+uint8_t ob_get_player_sequence(void)
+{
+ return ob.seq_player;
+}
+
diff --git a/src/outputBuffer_ob_send.h b/src/outputBuffer_ob_send.h
new file mode 100644
index 00000000..f27a5c0d
--- /dev/null
+++ b/src/outputBuffer_ob_send.h
@@ -0,0 +1,133 @@
+/*
+ * This file only contains ob_send() and private functions
+ * needed to implement ob_send()
+ */
+
+/*
+ * This is one of two places where dc.thread can block,
+ * the other is readFromInputStream
+ */
+static enum dc_action await_buffer_space(void)
+{
+ assert(pthread_equal(pthread_self(), dc.thread));
+ /* DEBUG("Waiting for buffer space\n"); */
+ assert(dc.state != DC_STATE_STOP);
+
+ dc_halt();
+ /* DEBUG("done waiting for buffer space\n"); */
+ return dc.action;
+}
+
+/* This will modify its input */
+static void do_audio_conversion(void **data, size_t *len)
+{
+ size_t newlen;
+
+ assert(pthread_equal(pthread_self(), dc.thread));
+ newlen = pcm_sizeOfConvBuffer(&dc.audio_format, *len, &ob.audio_format);
+ if (newlen > ob.conv_buf_len) {
+ ob.conv_buf = xrealloc(ob.conv_buf, newlen);
+ ob.conv_buf_len = newlen;
+ }
+ *len = pcm_convertAudioFormat(&dc.audio_format, *data, *len,
+ &ob.audio_format, ob.conv_buf,
+ &ob.conv_state);
+ *data = ob.conv_buf;
+}
+
+static void ensure_audio_format_sanity(void **data, size_t *len)
+{
+ static uint8_t seq_last;
+
+ assert(pthread_equal(pthread_self(), dc.thread));
+ if (mpd_unlikely(seq_last != ob.seq_decoder)) {
+ seq_last = ob.seq_decoder;
+ if (cmpAudioFormat(&dc.audio_format, &ob.audio_format))
+ getOutputAudioFormat(&dc.audio_format,
+ &ob.audio_format);
+ }
+ if (cmpAudioFormat(&ob.audio_format, &dc.audio_format))
+ do_audio_conversion(data, len);
+}
+
+static void start_playback(void)
+{
+ assert(pthread_equal(pthread_self(), dc.thread));
+ if (mpd_unlikely(ob.state == OB_STATE_STOP &&
+ player_errno == PLAYER_ERROR_NONE)) {
+ ob_trigger_action(OB_ACTION_PLAY);
+ }
+}
+
+enum dc_action
+ob_send(void *data, size_t len,
+ float decode_time, uint16_t bit_rate, ReplayGainInfo * rgi)
+{
+ struct rbvec vec[2];
+ struct ob_chunk *c;
+ size_t idx;
+ size_t i, j;
+
+ assert(pthread_equal(pthread_self(), dc.thread));
+
+ ensure_audio_format_sanity(&data, &len);
+
+ if (rgi && (replayGainState != REPLAYGAIN_OFF))
+ doReplayGain(rgi, data, len, &ob.audio_format);
+ else if (normalizationEnabled)
+ normalizeData(data, len, &ob.audio_format);
+
+ while (1) {
+ /* full buffer, loop check in case of spurious wakeups */
+ while (!ringbuf_get_write_vector(ob.index, vec)) {
+ enum dc_action rv = await_buffer_space();
+ if (rv != DC_ACTION_NONE)
+ return rv;
+ }
+
+ for (i = 0; i < ARRAY_SIZE(vec); i++) {
+ for (j = 0; j < vec[i].len; j++) {
+ size_t c_len;
+ idx = vec[i].base - ob.index->buf + j;
+ c = &(ob.chunks[idx]);
+
+ if (!c->len) { /* populate empty chunk */
+ c->seq = ob.seq_decoder;
+ c->time = decode_time;
+ c->bit_rate = bit_rate;
+ c_len = len > CHUNK_SIZE ? CHUNK_SIZE
+ : len;
+ c->len = (uint16_t)c_len;
+ memcpy(c->data, data, c_len);
+ } else { /* partially filled chunk */
+ size_t max = CHUNK_SIZE - c->len;
+ assert(c->seq == ob.seq_decoder);
+ c_len = len > max ? max : len;
+ assert(c_len <= CHUNK_SIZE);
+ memcpy(c->data + c->len, data, c_len);
+ c->len += c_len;
+ assert(c->len <= CHUNK_SIZE);
+ }
+
+ /*
+ * feed ob.thread ASAP, otherwise ob.thread
+ * will just play silence
+ */
+ if (c->len == CHUNK_SIZE)
+ ringbuf_write_advance(ob.index, 1);
+
+ assert(len >= c_len);
+ len -= c_len;
+ if (!len) {
+ start_playback();
+ return dc.action;
+ }
+ data = (unsigned char *)data + c_len;
+ }
+ }
+ }
+ assert(__FILE__ && __LINE__ && "We should never get here" && 0);
+ return DC_ACTION_NONE;
+}
+
+
diff --git a/src/pcm_utils.c b/src/pcm_utils.c
index 653df856..705f1499 100644
--- a/src/pcm_utils.c
+++ b/src/pcm_utils.c
@@ -35,8 +35,8 @@ pcm_dither(void)
* Check if the value is within the range of the provided bit size,
* and caps it if necessary.
*/
-static mpd_sint32
-pcm_range(mpd_sint32 sample, unsigned bits)
+static int32_t
+pcm_range(int32_t sample, unsigned bits)
{
if (mpd_unlikely(sample < (-1 << (bits - 1))))
return -1 << (bits - 1);
@@ -49,9 +49,9 @@ void pcm_volumeChange(char *buffer, int bufferSize,
const struct audio_format *format,
int volume)
{
- mpd_sint32 temp32;
- mpd_sint8 *buffer8 = (mpd_sint8 *) buffer;
- mpd_sint16 *buffer16 = (mpd_sint16 *) buffer;
+ int32_t temp32;
+ int8_t *buffer8 = (int8_t *) buffer;
+ int16_t *buffer16 = (int16_t *) buffer;
if (volume >= 1000)
return;
@@ -96,11 +96,11 @@ static void pcm_add(char *buffer1, const char *buffer2, size_t bufferSize1,
size_t bufferSize2, int vol1, int vol2,
const struct audio_format *format)
{
- mpd_sint32 temp32;
- mpd_sint8 *buffer8_1 = (mpd_sint8 *) buffer1;
- const mpd_sint8 *buffer8_2 = (const mpd_sint8 *) buffer2;
- mpd_sint16 *buffer16_1 = (mpd_sint16 *) buffer1;
- const mpd_sint16 *buffer16_2 = (const mpd_sint16 *) buffer2;
+ int32_t temp32;
+ int8_t *buffer8_1 = (int8_t *) buffer1;
+ const int8_t *buffer8_2 = (const int8_t *) buffer2;
+ int16_t *buffer16_1 = (int16_t *) buffer1;
+ const int16_t *buffer16_2 = (const int16_t *) buffer2;
switch (format->bits) {
case 16:
@@ -195,9 +195,9 @@ out:
#endif
#ifdef HAVE_LIBSAMPLERATE
-static size_t pcm_convertSampleRate(mpd_sint8 channels, mpd_uint32 inSampleRate,
+static size_t pcm_convertSampleRate(int8_t channels, uint32_t inSampleRate,
const char *inBuffer, size_t inSize,
- mpd_uint32 outSampleRate, char *outBuffer,
+ uint32_t outSampleRate, char *outBuffer,
size_t outSize, ConvState *convState)
{
static int convalgo = -1;
@@ -271,19 +271,19 @@ static size_t pcm_convertSampleRate(mpd_sint8 channels, mpd_uint32 inSampleRate,
}
#else /* !HAVE_LIBSAMPLERATE */
/* resampling code blatantly ripped from ESD */
-static size_t pcm_convertSampleRate(mpd_sint8 channels, mpd_uint32 inSampleRate,
+static size_t pcm_convertSampleRate(int8_t channels, uint32_t inSampleRate,
const char *inBuffer,
mpd_unused size_t inSize,
- mpd_uint32 outSampleRate, char *outBuffer,
+ uint32_t outSampleRate, char *outBuffer,
size_t outSize,
mpd_unused ConvState *convState)
{
- mpd_uint32 rd_dat = 0;
- mpd_uint32 wr_dat = 0;
- mpd_sint16 *in = (mpd_sint16 *)inBuffer;
- mpd_sint16 *out = (mpd_sint16 *)outBuffer;
- mpd_uint32 nlen = outSize / 2;
- mpd_sint16 lsample, rsample;
+ uint32_t rd_dat = 0;
+ uint32_t wr_dat = 0;
+ int16_t *in = (int16_t *)inBuffer;
+ int16_t *out = (int16_t *)outBuffer;
+ uint32_t nlen = outSize / 2;
+ int16_t lsample, rsample;
switch (channels) {
case 1:
@@ -313,14 +313,14 @@ static size_t pcm_convertSampleRate(mpd_sint8 channels, mpd_uint32 inSampleRate,
}
#endif /* !HAVE_LIBSAMPLERATE */
-static char *pcm_convertChannels(mpd_sint8 channels, const char *inBuffer,
+static char *pcm_convertChannels(int8_t channels, const char *inBuffer,
size_t inSize, size_t *outSize)
{
static char *buf;
static size_t len;
char *outBuffer = NULL;
- const mpd_sint16 *in;
- mpd_sint16 *out;
+ const int16_t *in;
+ int16_t *out;
int inSamples, i;
switch (channels) {
@@ -334,8 +334,8 @@ static char *pcm_convertChannels(mpd_sint8 channels, const char *inBuffer,
outBuffer = buf;
inSamples = inSize >> 1;
- in = (const mpd_sint16 *)inBuffer;
- out = (mpd_sint16 *)outBuffer;
+ in = (const int16_t *)inBuffer;
+ out = (int16_t *)outBuffer;
for (i = 0; i < inSamples; i++) {
*out++ = *in;
*out++ = *in++;
@@ -352,8 +352,8 @@ static char *pcm_convertChannels(mpd_sint8 channels, const char *inBuffer,
outBuffer = buf;
inSamples = inSize >> 2;
- in = (const mpd_sint16 *)inBuffer;
- out = (mpd_sint16 *)outBuffer;
+ in = (const int16_t *)inBuffer;
+ out = (int16_t *)outBuffer;
for (i = 0; i < inSamples; i++) {
*out = (*in++) / 2;
*out++ += (*in++) / 2;
@@ -367,14 +367,14 @@ static char *pcm_convertChannels(mpd_sint8 channels, const char *inBuffer,
return outBuffer;
}
-static const char *pcm_convertTo16bit(mpd_sint8 bits, const char *inBuffer,
+static const char *pcm_convertTo16bit(int8_t bits, const char *inBuffer,
size_t inSize, size_t *outSize)
{
static char *buf;
static size_t len;
char *outBuffer = NULL;
- const mpd_sint8 *in;
- mpd_sint16 *out;
+ const int8_t *in;
+ int16_t *out;
size_t i;
switch (bits) {
@@ -386,8 +386,8 @@ static const char *pcm_convertTo16bit(mpd_sint8 bits, const char *inBuffer,
}
outBuffer = buf;
- in = (const mpd_sint8 *)inBuffer;
- out = (mpd_sint16 *)outBuffer;
+ in = (const int8_t *)inBuffer;
+ out = (int16_t *)outBuffer;
for (i = 0; i < inSize; i++)
*out++ = (*in++) << 8;
diff --git a/src/pcm_utils.h b/src/pcm_utils.h
index 414691f5..98589ce4 100644
--- a/src/pcm_utils.h
+++ b/src/pcm_utils.h
@@ -36,9 +36,9 @@ typedef struct _ConvState {
SRC_DATA data;
size_t dataInSize;
size_t dataOutSize;
- mpd_sint8 lastChannels;
- mpd_uint32 lastInSampleRate;
- mpd_uint32 lastOutSampleRate;
+ int8_t lastChannels;
+ uint32_t lastInSampleRate;
+ uint32_t lastOutSampleRate;
#endif
/* Strict C99 doesn't allow empty structs */
int error;
diff --git a/src/replayGain.c b/src/replayGain.c
index 27caac1b..80a102c0 100644
--- a/src/replayGain.c
+++ b/src/replayGain.c
@@ -108,9 +108,9 @@ void freeReplayGainInfo(ReplayGainInfo * info)
void doReplayGain(ReplayGainInfo * info, char *buffer, int bufferSize,
const struct audio_format *format)
{
- mpd_sint16 *buffer16;
- mpd_sint8 *buffer8;
- mpd_sint32 temp32;
+ int16_t *buffer16;
+ int8_t *buffer8;
+ int32_t temp32;
float scale;
if (replayGainState == REPLAYGAIN_OFF || !info)
@@ -136,8 +136,8 @@ void doReplayGain(ReplayGainInfo * info, char *buffer, int bufferSize,
if (info->scale <= 1.01 && info->scale >= 0.99)
return;
- buffer16 = (mpd_sint16 *) buffer;
- buffer8 = (mpd_sint8 *) buffer;
+ buffer16 = (int16_t *) buffer;
+ buffer8 = (int8_t *) buffer;
scale = info->scale;
diff --git a/src/tag.c b/src/tag.c
index 376e6f27..dc203747 100644
--- a/src/tag.c
+++ b/src/tag.c
@@ -52,7 +52,7 @@ const char *mpdTagItemKeys[TAG_NUM_OF_ITEM_TYPES] = {
"Disc"
};
-mpd_sint8 ignoreTagItems[TAG_NUM_OF_ITEM_TYPES];
+int8_t ignoreTagItems[TAG_NUM_OF_ITEM_TYPES];
static size_t items_size(const struct tag *tag)
{
diff --git a/src/tag.h b/src/tag.h
index a6cb4a78..121b2ae1 100644
--- a/src/tag.h
+++ b/src/tag.h
@@ -48,7 +48,7 @@ struct tag_item {
struct tag {
int time;
struct tag_item **items;
- mpd_uint8 numOfItems;
+ uint8_t numOfItems;
};
struct tag *tag_ape_load(const char *file);
diff --git a/src/tag_internal.h b/src/tag_internal.h
index 165a9d66..eed3e079 100644
--- a/src/tag_internal.h
+++ b/src/tag_internal.h
@@ -19,6 +19,6 @@
#ifndef TAG_INTERNAL_H
#define TAG_INTERNAL_H
-extern mpd_sint8 ignoreTagItems[TAG_NUM_OF_ITEM_TYPES];
+extern int8_t ignoreTagItems[TAG_NUM_OF_ITEM_TYPES];
#endif