diff options
| author | Max Kellermann <max@duempel.org> | 2008-04-12 04:18:28 +0000 |
|---|---|---|
| committer | Eric Wong <normalperson@yhbt.net> | 2008-04-12 04:18:28 +0000 |
| commit | 0c46dd68318ea20cb65adae433c3f8793ce73435 (patch) | |
| tree | 58c231cb4c04e8545538f7d07f3ed0206ef8f06b /src/outputBuffer.c | |
| parent | 20c4638a2eb28bd66cd4494b706f113f60a2c0e2 (diff) | |
| download | mpd-0c46dd68318ea20cb65adae433c3f8793ce73435.tar.gz mpd-0c46dd68318ea20cb65adae433c3f8793ce73435.tar.xz mpd-0c46dd68318ea20cb65adae433c3f8793ce73435.zip | |
don't use short chunk numbers
Don't be mean with integer sizes. Although we will probably never
have more than 32k buffered chunks, we should use 32 bit integers for
addressing them. We do not save very much (some of the saved space is
eaten by alignment anyway), but we save at least one assembler
instruction for converting short to int.
This change requires some more explicit casts, because gcc was less
picky when comparing short with a full int.
git-svn-id: https://svn.musicpd.org/mpd/trunk@7313 09075e82-0dd4-0310-85a5-a0d7c8717e4f
Diffstat (limited to '')
| -rw-r--r-- | src/outputBuffer.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/outputBuffer.c b/src/outputBuffer.c index 4995fa9a5..8e2780eb2 100644 --- a/src/outputBuffer.c +++ b/src/outputBuffer.c @@ -53,7 +53,7 @@ static inline unsigned successor(const OutputBuffer * cb, unsigned i) void flushOutputBuffer(OutputBuffer * cb) { - if (cb->currentChunk == cb->end) { + if (cb->currentChunk == (int)cb->end) { cb->end = successor(cb, cb->end); cb->currentChunk = -1; } @@ -125,7 +125,7 @@ static int tailChunk(OutputBuffer * cb, InputStream * inStream, unsigned int next; OutputBufferChunk *chunk; - if (cb->currentChunk == cb->end) + if (cb->currentChunk == (int)cb->end) return cb->currentChunk; next = successor(cb, cb->end); |