diff options
author | Max Kellermann <max@duempel.org> | 2008-08-25 15:49:06 +0200 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2008-08-25 15:49:06 +0200 |
commit | 35918c990163b2e4350156aba11c50ebd2807ef1 (patch) | |
tree | 276ce3ee84d5a3b0b799081538887e05b7e976e0 /src/ringbuf.c | |
parent | f9f70860622613686e6ac0bf7ebd448f437d92a7 (diff) | |
download | mpd-35918c990163b2e4350156aba11c50ebd2807ef1.tar.gz mpd-35918c990163b2e4350156aba11c50ebd2807ef1.tar.xz mpd-35918c990163b2e4350156aba11c50ebd2807ef1.zip |
fixed ringbuf.c warnings
Fix a "signed/unsigned comparison warning", and several void pointer
math warnings.
Diffstat (limited to 'src/ringbuf.c')
-rw-r--r-- | src/ringbuf.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/ringbuf.c b/src/ringbuf.c index ed46c3beb..20a09b326 100644 --- a/src/ringbuf.c +++ b/src/ringbuf.c @@ -36,7 +36,7 @@ struct ringbuf *ringbuf_create(size_t sz) struct ringbuf *rb = xmalloc(sizeof(struct ringbuf)); size_t power_of_two; - for (power_of_two = 1; 1 << power_of_two < sz; power_of_two++) + for (power_of_two = 1; (size_t)(1 << power_of_two) < sz; power_of_two++) /* next power_of_two... */; rb->size = 1 << power_of_two; @@ -137,7 +137,7 @@ size_t ringbuf_read(struct ringbuf * rb, void *dest, size_t cnt) ringbuf_read_advance(rb, n1); if (n2) { - memcpy(dest + n1, rb->buf + rb->read_ptr, n2); + memcpy((char*)dest + n1, rb->buf + rb->read_ptr, n2); ringbuf_read_advance(rb, n2); } @@ -175,7 +175,7 @@ size_t ringbuf_peek(struct ringbuf * rb, void *dest, size_t cnt) advance_ptr(tmp_read_ptr, n1, rb->size_mask); if (n2) { - memcpy(dest + n1, rb->buf + tmp_read_ptr, n2); + memcpy((char*)dest + n1, rb->buf + tmp_read_ptr, n2); advance_ptr(tmp_read_ptr, n2, rb->size_mask); } @@ -211,7 +211,7 @@ size_t ringbuf_write(struct ringbuf * rb, const void *src, size_t cnt) ringbuf_write_advance(rb, n1); if (n2) { - memcpy(rb->buf + rb->write_ptr, src + n1, n2); + memcpy(rb->buf + rb->write_ptr, (const char*)src + n1, n2); ringbuf_write_advance(rb, n2); } |