diff options
author | Eric Wong <normalperson@yhbt.net> | 2006-07-15 13:42:57 +0000 |
---|---|---|
committer | Eric Wong <normalperson@yhbt.net> | 2006-07-15 13:42:57 +0000 |
commit | b4df8b8f880648e63781395e1a350f258b809a18 (patch) | |
tree | 3e26b8faf86cf68f35506eb3e59b73d9477211e7 /src | |
parent | 563e4c95fdea05dc162442d6b3a66e3fa0a13a48 (diff) |
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
Diffstat (limited to 'src')
-rw-r--r-- | src/audio.c | 2 | ||||
-rw-r--r-- | src/audioOutputs/audioOutput_alsa.c | 2 | ||||
-rw-r--r-- | src/dbUtils.c | 4 | ||||
-rw-r--r-- | src/inputPlugins/_flac_common.h | 1 | ||||
-rw-r--r-- | src/inputPlugins/mpc_plugin.c | 3 | ||||
-rw-r--r-- | src/inputPlugins/oggvorbis_plugin.c | 2 | ||||
-rw-r--r-- | src/tag.c | 2 |
7 files changed, 9 insertions, 7 deletions
diff --git a/src/audio.c b/src/audio.c index d0cd53da3..da2a20c80 100644 --- a/src/audio.c +++ b/src/audio.c @@ -251,7 +251,7 @@ int isCurrentAudioFormat(AudioFormat * audioFormat) { return 1; } -inline void syncAudioDevicesEnabledArrays() { +static void syncAudioDevicesEnabledArrays(void) { int i; memcpy(myAudioDevicesEnabled, pdAudioDevicesEnabled,AUDIO_MAX_DEVICES); diff --git a/src/audioOutputs/audioOutput_alsa.c b/src/audioOutputs/audioOutput_alsa.c index 4484d2ca4..976b72d31 100644 --- a/src/audioOutputs/audioOutput_alsa.c +++ b/src/audioOutputs/audioOutput_alsa.c @@ -283,7 +283,7 @@ fail: return -1; } -inline static int alsa_errorRecovery(AlsaData * ad, int err) { +static int alsa_errorRecovery(AlsaData * ad, int err) { if(err == -EPIPE) { DEBUG("Underrun on alsa device \"%s\"\n", ad->device); } diff --git a/src/dbUtils.c b/src/dbUtils.c index ecc8b8878..236d0d734 100644 --- a/src/dbUtils.c +++ b/src/dbUtils.c @@ -147,7 +147,7 @@ static int printSongInDirectory(FILE * fp, Song * song, void * data) { return 0; } -static inline int strstrSearchTag(Song * song, int type, char * str) { +static int strstrSearchTag(Song * song, int type, char * str) { int i; char * dup; int ret = 0; @@ -224,7 +224,7 @@ int searchForSongsIn(FILE * fp, char * name, int numItems, return ret; } -static inline int tagItemFoundAndMatches(Song * song, int type, char * str) { +static int tagItemFoundAndMatches(Song * song, int type, char * str) { int i; if(type == LOCATE_TAG_FILE_TYPE) { diff --git a/src/inputPlugins/_flac_common.h b/src/inputPlugins/_flac_common.h index 88fa4da58..2834d3524 100644 --- a/src/inputPlugins/_flac_common.h +++ b/src/inputPlugins/_flac_common.h @@ -60,6 +60,7 @@ void flac_error_common_cb( const char * plugin, MpdTag * copyVorbisCommentBlockToMpdTag(const FLAC__StreamMetadata * block, MpdTag * tag); +/* keep this inlined, this is just macro but prettier :) */ static inline int flacSendChunk(FlacData * data) { if (sendDataToOutputBuffer(data->cb, NULL, data->dc, 1, data->chunk, diff --git a/src/inputPlugins/mpc_plugin.c b/src/inputPlugins/mpc_plugin.c index 30878fc16..253ed0891 100644 --- a/src/inputPlugins/mpc_plugin.c +++ b/src/inputPlugins/mpc_plugin.c @@ -82,7 +82,8 @@ static mpc_int32_t mpc_getsize_cb(void * vdata) { 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 */ mpd_sint32 val; diff --git a/src/inputPlugins/oggvorbis_plugin.c b/src/inputPlugins/oggvorbis_plugin.c index 30034af69..3c699cef2 100644 --- a/src/inputPlugins/oggvorbis_plugin.c +++ b/src/inputPlugins/oggvorbis_plugin.c @@ -103,7 +103,7 @@ long ogg_tell_cb(void * vdata) { 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); if(strncasecmp(comment, needle, len) == 0 && *(comment+len) == '=') { @@ -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 i = tag->numOfItems; |