diff options
author | Max Kellermann <max@duempel.org> | 2008-11-02 16:57:37 +0100 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2008-11-02 16:57:37 +0100 |
commit | 39bf84aa9f2ff43aa3694fe9a5b501573c7746f5 (patch) | |
tree | 2d9534a658595537537b05f3c1dbfc494c5b61a0 | |
parent | 5347cca29d0b6df888122d09ec05101d24a227be (diff) | |
download | mpd-39bf84aa9f2ff43aa3694fe9a5b501573c7746f5.tar.gz mpd-39bf84aa9f2ff43aa3694fe9a5b501573c7746f5.tar.xz mpd-39bf84aa9f2ff43aa3694fe9a5b501573c7746f5.zip |
music_pipe: renamed "size" to "num_chunks"
The name "num_chunks" expresses the meaning of the variable better.
-rw-r--r-- | src/pipe.c | 20 | ||||
-rw-r--r-- | src/pipe.h | 5 |
2 files changed, 12 insertions, 13 deletions
diff --git a/src/pipe.c b/src/pipe.c index 38a016958..9d781961a 100644 --- a/src/pipe.c +++ b/src/pipe.c @@ -32,7 +32,7 @@ music_pipe_init(unsigned int size, struct notify *notify) assert(size > 0); music_pipe.chunks = g_new(struct music_chunk, size); - music_pipe.size = size; + music_pipe.num_chunks = size; music_pipe.begin = 0; music_pipe.end = 0; music_pipe.lazy = false; @@ -55,10 +55,10 @@ void music_pipe_clear(void) /** return the index of the chunk after i */ static inline unsigned successor(unsigned i) { - assert(i <= music_pipe.size); + assert(i <= music_pipe.num_chunks); ++i; - return i == music_pipe.size ? 0 : i; + return i == music_pipe.num_chunks ? 0 : i; } /** @@ -69,7 +69,7 @@ static void output_buffer_expand(unsigned i) { int was_empty = music_pipe.notify != NULL && (!music_pipe.lazy || music_pipe_is_empty()); - assert(i == (music_pipe.end + 1) % music_pipe.size); + assert(i == (music_pipe.end + 1) % music_pipe.num_chunks); assert(i != music_pipe.end); music_pipe.end = i; @@ -105,7 +105,7 @@ void music_pipe_set_lazy(bool lazy) void music_pipe_shift(void) { assert(music_pipe.begin != music_pipe.end); - assert(music_pipe.begin < music_pipe.size); + assert(music_pipe.begin < music_pipe.num_chunks); music_pipe.begin = successor(music_pipe.begin); } @@ -115,7 +115,7 @@ unsigned int music_pipe_relative(const unsigned i) if (i >= music_pipe.begin) return i - music_pipe.begin; else - return i + music_pipe.size - music_pipe.begin; + return i + music_pipe.num_chunks - music_pipe.begin; } unsigned music_pipe_available(void) @@ -129,13 +129,13 @@ int music_pipe_absolute(const unsigned relative) max = music_pipe.end; if (max < music_pipe.begin) - max += music_pipe.size; + max += music_pipe.num_chunks; i = (unsigned)music_pipe.begin + relative; if (i >= max) return -1; - if (i >= music_pipe.size) - i -= music_pipe.size; + if (i >= music_pipe.num_chunks) + i -= music_pipe.num_chunks; return (int)i; } @@ -143,7 +143,7 @@ int music_pipe_absolute(const unsigned relative) struct music_chunk * music_pipe_get_chunk(const unsigned i) { - assert(i < music_pipe.size); + assert(i < music_pipe.num_chunks); return &music_pipe.chunks[i]; } diff --git a/src/pipe.h b/src/pipe.h index 51c8ed3f3..bfc1a7974 100644 --- a/src/pipe.h +++ b/src/pipe.h @@ -41,8 +41,7 @@ struct music_chunk { */ struct music_pipe { struct music_chunk *chunks; - - unsigned int size; + unsigned num_chunks; /** the index of the first decoded chunk */ unsigned int volatile begin; @@ -79,7 +78,7 @@ void music_pipe_set_lazy(bool lazy); static inline unsigned music_pipe_size(void) { - return music_pipe.size; + return music_pipe.num_chunks; } /** is the buffer empty? */ |