From b4df8b8f880648e63781395e1a350f258b809a18 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sat, 15 Jul 2006 13:42:57 +0000 Subject: 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 --- src/inputPlugins/_flac_common.h | 1 + src/inputPlugins/mpc_plugin.c | 3 ++- src/inputPlugins/oggvorbis_plugin.c | 2 +- 3 files changed, 4 insertions(+), 2 deletions(-) (limited to 'src/inputPlugins') 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) == '=') { -- cgit v1.2.3