Commit b4df8b8f authored by Eric Wong's avatar Eric Wong

De-inline non-trivial, non-performance-critical functions

Functions that should stay inlined should have an explanation attached to them. git-svn-id: https://svn.musicpd.org/mpd/trunk@4355 09075e82-0dd4-0310-85a5-a0d7c8717e4f
parent 563e4c95
...@@ -251,7 +251,7 @@ int isCurrentAudioFormat(AudioFormat * audioFormat) { ...@@ -251,7 +251,7 @@ int isCurrentAudioFormat(AudioFormat * audioFormat) {
return 1; return 1;
} }
inline void syncAudioDevicesEnabledArrays() { static void syncAudioDevicesEnabledArrays(void) {
int i; int i;
memcpy(myAudioDevicesEnabled, pdAudioDevicesEnabled,AUDIO_MAX_DEVICES); memcpy(myAudioDevicesEnabled, pdAudioDevicesEnabled,AUDIO_MAX_DEVICES);
......
...@@ -283,7 +283,7 @@ fail: ...@@ -283,7 +283,7 @@ fail:
return -1; return -1;
} }
inline static int alsa_errorRecovery(AlsaData * ad, int err) { static int alsa_errorRecovery(AlsaData * ad, int err) {
if(err == -EPIPE) { if(err == -EPIPE) {
DEBUG("Underrun on alsa device \"%s\"\n", ad->device); DEBUG("Underrun on alsa device \"%s\"\n", ad->device);
} }
......
...@@ -147,7 +147,7 @@ static int printSongInDirectory(FILE * fp, Song * song, void * data) { ...@@ -147,7 +147,7 @@ static int printSongInDirectory(FILE * fp, Song * song, void * data) {
return 0; return 0;
} }
static inline int strstrSearchTag(Song * song, int type, char * str) { static int strstrSearchTag(Song * song, int type, char * str) {
int i; int i;
char * dup; char * dup;
int ret = 0; int ret = 0;
...@@ -224,7 +224,7 @@ int searchForSongsIn(FILE * fp, char * name, int numItems, ...@@ -224,7 +224,7 @@ int searchForSongsIn(FILE * fp, char * name, int numItems,
return ret; return ret;
} }
static inline int tagItemFoundAndMatches(Song * song, int type, char * str) { static int tagItemFoundAndMatches(Song * song, int type, char * str) {
int i; int i;
if(type == LOCATE_TAG_FILE_TYPE) { if(type == LOCATE_TAG_FILE_TYPE) {
......
...@@ -60,6 +60,7 @@ void flac_error_common_cb( const char * plugin, ...@@ -60,6 +60,7 @@ void flac_error_common_cb( const char * plugin,
MpdTag * copyVorbisCommentBlockToMpdTag(const FLAC__StreamMetadata * block, MpdTag * copyVorbisCommentBlockToMpdTag(const FLAC__StreamMetadata * block,
MpdTag * tag); MpdTag * tag);
/* keep this inlined, this is just macro but prettier :) */
static inline int flacSendChunk(FlacData * data) static inline int flacSendChunk(FlacData * data)
{ {
if (sendDataToOutputBuffer(data->cb, NULL, data->dc, 1, data->chunk, if (sendDataToOutputBuffer(data->cb, NULL, data->dc, 1, data->chunk,
......
...@@ -82,7 +82,8 @@ static mpc_int32_t mpc_getsize_cb(void * vdata) { ...@@ -82,7 +82,8 @@ static mpc_int32_t mpc_getsize_cb(void * vdata) {
return data->inStream->size; return data->inStream->size;
} }
inline mpd_sint16 convertSample(MPC_SAMPLE_FORMAT sample) { /* this _looks_ performance-critical, don't de-inline -- eric */
static inline mpd_sint16 convertSample(MPC_SAMPLE_FORMAT sample) {
/* only doing 16-bit audio for now */ /* only doing 16-bit audio for now */
mpd_sint32 val; mpd_sint32 val;
......
...@@ -103,7 +103,7 @@ long ogg_tell_cb(void * vdata) { ...@@ -103,7 +103,7 @@ long ogg_tell_cb(void * vdata) {
return (long)(data->inStream->offset); return (long)(data->inStream->offset);
} }
static inline char * ogg_parseComment(char * comment, char * needle) { static char * ogg_parseComment(char * comment, char * needle) {
int len = strlen(needle); int len = strlen(needle);
if(strncasecmp(comment, needle, len) == 0 && *(comment+len) == '=') { if(strncasecmp(comment, needle, len) == 0 && *(comment+len) == '=') {
......
...@@ -560,7 +560,7 @@ int mpdTagsAreEqual(MpdTag * tag1, MpdTag * tag2) { ...@@ -560,7 +560,7 @@ int mpdTagsAreEqual(MpdTag * tag1, MpdTag * tag2) {
} \ } \
} }
inline static void appendToTagItems(MpdTag * tag, int type, char * value, static void appendToTagItems(MpdTag * tag, int type, char * value,
int len) int len)
{ {
int i = tag->numOfItems; int i = tag->numOfItems;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment