diff options
author | wm4 <wm4@nowhere> | 2013-03-19 01:27:48 +0100 |
---|---|---|
committer | wm4 <wm4@nowhere> | 2013-03-19 01:27:48 +0100 |
commit | b242aa366bc10b0dd0504f3c3c916e4e61e4f81b (patch) | |
tree | 6b68bceba25a6cfe27b47ab77564c70dcd832027 /stream | |
parent | 7ba66758476c81dd2c86635b9052bd7ee67f6047 (diff) | |
download | mpv-b242aa366bc10b0dd0504f3c3c916e4e61e4f81b.tar.bz2 mpv-b242aa366bc10b0dd0504f3c3c916e4e61e4f81b.tar.xz |
stream: silence clang empty statement warnings
clang printed warnings like:
stream/stream.c:692:65: warning: if statement has empty body [-Wempty-body]
GET_UTF16(c, src < end - 1 ? get_le16_inc(&src) : 0,;
This macro expands to "if(cond) ;". Replace it with an empty statement
that doesn't lead to a clang warning.
Diffstat (limited to 'stream')
-rw-r--r-- | stream/stream.c | 20 |
1 files changed, 8 insertions, 12 deletions
diff --git a/stream/stream.c b/stream/stream.c index d0bd0b58d6..75c7d4fb5c 100644 --- a/stream/stream.c +++ b/stream/stream.c @@ -649,16 +649,14 @@ static const uint8_t *find_newline(const uint8_t *buf, int len, int utf16) return (uint8_t *)memchr(buf, '\n', len); case 1: while (buf < end - 1) { - GET_UTF16(c, buf < end - 1 ? get_le16_inc(&buf) : 0, return NULL; - ) + GET_UTF16(c, buf < end - 1 ? get_le16_inc(&buf) : 0, return NULL;) if (buf <= end && c == '\n') return buf - 1; } break; case 2: while (buf < end - 1) { - GET_UTF16(c, buf < end - 1 ? get_be16_inc(&buf) : 0, return NULL; - ) + GET_UTF16(c, buf < end - 1 ? get_be16_inc(&buf) : 0, return NULL;) if (buf <= end && c == '\n') return buf - 1; } @@ -667,6 +665,8 @@ static const uint8_t *find_newline(const uint8_t *buf, int len, int utf16) return NULL; } +#define EMPTY_STMT do{}while(0); + /** * Copy a number of bytes, converting to UTF-8 if input is UTF-16 * \param dst buffer to copy to @@ -691,20 +691,16 @@ static int copy_characters(uint8_t *dst, int dstsize, case 1: while (src < end - 1 && dst_end - dst > 8) { uint8_t tmp; - GET_UTF16(c, src < end - 1 ? get_le16_inc(&src) : 0,; - ) - PUT_UTF8(c, tmp, *dst++ = tmp; - ) + GET_UTF16(c, src < end - 1 ? get_le16_inc(&src) : 0, EMPTY_STMT) + PUT_UTF8(c, tmp, *dst++ = tmp; EMPTY_STMT) } *len -= end - src; return dstsize - (dst_end - dst); case 2: while (src < end - 1 && dst_end - dst > 8) { uint8_t tmp; - GET_UTF16(c, src < end - 1 ? get_be16_inc(&src) : 0,; - ) - PUT_UTF8(c, tmp, *dst++ = tmp; - ) + GET_UTF16(c, src < end - 1 ? get_be16_inc(&src) : 0, EMPTY_STMT) + PUT_UTF8(c, tmp, *dst++ = tmp; EMPTY_STMT) } *len -= end - src; return dstsize - (dst_end - dst); |