diff options
| author | kinichiro <kinichiro.inoguchi@gmail.com> | 2018-01-06 00:28:49 +0900 |
|---|---|---|
| committer | kinichiro <kinichiro.inoguchi@gmail.com> | 2018-01-06 00:28:49 +0900 |
| commit | 1f4bfb9a4db12b0b02a63599cecab5d2b0a6458f (patch) | |
| tree | ee6014e63a93f81c6ccc32b60074be313d1d6902 /include | |
| parent | 52c39d42ea083506575ab75929dfa1edddd94c08 (diff) | |
| download | portable-1f4bfb9a4db12b0b02a63599cecab5d2b0a6458f.tar.gz portable-1f4bfb9a4db12b0b02a63599cecab5d2b0a6458f.tar.bz2 portable-1f4bfb9a4db12b0b02a63599cecab5d2b0a6458f.zip | |
Call va_end soon after using variable length variable
- This fix MSVC compiler warning C4702 "unreachable code".
Diffstat (limited to 'include')
| -rw-r--r-- | include/compat/err.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/include/compat/err.h b/include/compat/err.h index 3a5e01f..8b5b635 100644 --- a/include/compat/err.h +++ b/include/compat/err.h | |||
| @@ -34,9 +34,9 @@ err(int eval, const char *fmt, ...) | |||
| 34 | vfprintf(stderr, fmt, ap); | 34 | vfprintf(stderr, fmt, ap); |
| 35 | fprintf(stderr, ": "); | 35 | fprintf(stderr, ": "); |
| 36 | } | 36 | } |
| 37 | va_end(ap); | ||
| 37 | fprintf(stderr, "%s\n", strerror(sverrno)); | 38 | fprintf(stderr, "%s\n", strerror(sverrno)); |
| 38 | exit(eval); | 39 | exit(eval); |
| 39 | va_end(ap); | ||
| 40 | } | 40 | } |
| 41 | 41 | ||
| 42 | #if defined(_MSC_VER) | 42 | #if defined(_MSC_VER) |
| @@ -52,9 +52,9 @@ errx(int eval, const char *fmt, ...) | |||
| 52 | va_start(ap, fmt); | 52 | va_start(ap, fmt); |
| 53 | if (fmt != NULL) | 53 | if (fmt != NULL) |
| 54 | vfprintf(stderr, fmt, ap); | 54 | vfprintf(stderr, fmt, ap); |
| 55 | va_end(ap); | ||
| 55 | fprintf(stderr, "\n"); | 56 | fprintf(stderr, "\n"); |
| 56 | exit(eval); | 57 | exit(eval); |
| 57 | va_end(ap); | ||
| 58 | } | 58 | } |
| 59 | 59 | ||
| 60 | static inline void | 60 | static inline void |
| @@ -68,8 +68,8 @@ warn(const char *fmt, ...) | |||
| 68 | vfprintf(stderr, fmt, ap); | 68 | vfprintf(stderr, fmt, ap); |
| 69 | fprintf(stderr, ": "); | 69 | fprintf(stderr, ": "); |
| 70 | } | 70 | } |
| 71 | fprintf(stderr, "%s\n", strerror(sverrno)); | ||
| 72 | va_end(ap); | 71 | va_end(ap); |
| 72 | fprintf(stderr, "%s\n", strerror(sverrno)); | ||
| 73 | } | 73 | } |
| 74 | 74 | ||
| 75 | static inline void | 75 | static inline void |
| @@ -80,8 +80,8 @@ warnx(const char *fmt, ...) | |||
| 80 | va_start(ap, fmt); | 80 | va_start(ap, fmt); |
| 81 | if (fmt != NULL) | 81 | if (fmt != NULL) |
| 82 | vfprintf(stderr, fmt, ap); | 82 | vfprintf(stderr, fmt, ap); |
| 83 | fprintf(stderr, "\n"); | ||
| 84 | va_end(ap); | 83 | va_end(ap); |
| 84 | fprintf(stderr, "\n"); | ||
| 85 | } | 85 | } |
| 86 | 86 | ||
| 87 | #endif | 87 | #endif |
