diff options
author | Chris Peterson <cpeterson@mozilla.com> | 2016-03-26 08:19:28 (GMT) |
---|---|---|
committer | Jason Evans <jasone@canonware.com> | 2016-03-27 06:24:33 (GMT) |
commit | 0bc716ae27d1bd66faa8f165a2c4a4cf6bd8143f (patch) | |
tree | 7d06de12d64bd23679e73f0183f883a802d2b177 | |
parent | f3060284c521cc74e333c5ab3a6c8fc0648defb5 (diff) | |
download | jemalloc-0bc716ae27d1bd66faa8f165a2c4a4cf6bd8143f.zip jemalloc-0bc716ae27d1bd66faa8f165a2c4a4cf6bd8143f.tar.gz jemalloc-0bc716ae27d1bd66faa8f165a2c4a4cf6bd8143f.tar.bz2 |
Fix -Wunreachable-code warning in malloc_vsnprintf().
Variables s and slen are declared inside a switch statement, but outside
a case scope. clang reports these variable definitions as "unreachable",
though this is not really meaningful in this case. This is the only
-Wunreachable-code warning in jemalloc.
src/util.c:501:5 [-Wunreachable-code] code will never be executed
This resolves #364.
-rw-r--r-- | src/util.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -408,6 +408,8 @@ malloc_vsnprintf(char *str, size_t size, const char *format, va_list ap) int prec = -1; int width = -1; unsigned char len = '?'; + char *s; + size_t slen; f++; /* Flags. */ @@ -498,8 +500,6 @@ malloc_vsnprintf(char *str, size_t size, const char *format, va_list ap) } /* Conversion specifier. */ switch (*f) { - char *s; - size_t slen; case '%': /* %% */ APPEND_C(*f); |