oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

pcm_byteswap: use "end" pointer instead of buffer size

Browse Source
This commit is contained in:
Max Kellermann 2011-10-10 10:14:55 +02:00
parent 20c6159c04
commit 08a0bb756d
1 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/pcm_byteswap.c
View File

@ -31,13 +31,16 @@
const int16_t *pcm_byteswap_16(struct pcm_buffer *buffer, const int16_t *pcm_byteswap_16(struct pcm_buffer *buffer,
const int16_t *src, size_t len) const int16_t *src, size_t len)
{ {
unsigned i;
int16_t *buf = pcm_buffer_get(buffer, len); int16_t *buf = pcm_buffer_get(buffer, len);
assert(buf != NULL); assert(buf != NULL);
for (i = 0; i < len / 2; i++) const int16_t *src_end = src + len / sizeof(*src);
buf[i] = GUINT16_SWAP_LE_BE(src[i]); int16_t *dest = buf;
while (src < src_end) {
const int16_t x = *src++;
*dest++ = GUINT16_SWAP_LE_BE(x);
}
return buf; return buf;
} }
@ -45,13 +48,16 @@ const int16_t *pcm_byteswap_16(struct pcm_buffer *buffer,
const int32_t *pcm_byteswap_32(struct pcm_buffer *buffer, const int32_t *pcm_byteswap_32(struct pcm_buffer *buffer,
const int32_t *src, size_t len) const int32_t *src, size_t len)
{ {
unsigned i;
int32_t *buf = pcm_buffer_get(buffer, len); int32_t *buf = pcm_buffer_get(buffer, len);
assert(buf != NULL); assert(buf != NULL);
for (i = 0; i < len / 4; i++) const int32_t *src_end = src + len / sizeof(*src);
buf[i] = GUINT32_SWAP_LE_BE(src[i]); int32_t *dest = buf;
while (src < src_end) {
const int32_t x = *src++;
*dest++ = GUINT32_SWAP_LE_BE(x);
}
return buf; return buf;
} }