Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/external/bsd/jemalloc/include/jemalloc/internal use __format...



details:   https://anonhg.NetBSD.org/src/rev/bc08811318ae
branches:  trunk
changeset: 997350:bc08811318ae
user:      christos <christos%NetBSD.org@localhost>
date:      Mon Mar 04 19:40:57 2019 +0000

description:
use __format_arg__ to check arguments...

diffstat:

 external/bsd/jemalloc/include/jemalloc/internal/emitter.h |  33 ++++++++------
 1 files changed, 18 insertions(+), 15 deletions(-)

diffs (84 lines):

diff -r 643e248d8cbe -r bc08811318ae external/bsd/jemalloc/include/jemalloc/internal/emitter.h
--- a/external/bsd/jemalloc/include/jemalloc/internal/emitter.h Mon Mar 04 19:20:12 2019 +0000
+++ b/external/bsd/jemalloc/include/jemalloc/internal/emitter.h Mon Mar 04 19:40:57 2019 +0000
@@ -115,10 +115,11 @@
        }
 }
 
-static inline void
+static inline const char * __attribute__((__format_arg__(3)))
 emitter_gen_fmt(char *out_fmt, size_t out_size, const char *fmt_specifier,
     emitter_justify_t justify, int width) {
        size_t written;
+       fmt_specifier++;
        if (justify == emitter_justify_none) {
                written = malloc_snprintf(out_fmt, out_size,
                    "%%%s", fmt_specifier);
@@ -131,6 +132,7 @@
        }
        /* Only happens in case of bad format string, which *we* choose. */
        assert(written <  out_size);
+       return out_fmt;
 }
 
 /*
@@ -156,26 +158,27 @@
        char buf[BUF_SIZE];
 
 #define EMIT_SIMPLE(type, format)                                      \
-       emitter_gen_fmt(fmt, FMT_SIZE, format, justify, width);         \
-       emitter_printf(emitter, fmt, *(const type *)value);             \
+       emitter_printf(emitter,                                         \
+           emitter_gen_fmt(fmt, FMT_SIZE, format, justify, width),     \
+           *(const type *)value);
 
        switch (value_type) {
        case emitter_type_bool:
-               emitter_gen_fmt(fmt, FMT_SIZE, "s", justify, width);
-               emitter_printf(emitter, fmt, *(const bool *)value ?
-                   "true" : "false");
+               emitter_printf(emitter, 
+                   emitter_gen_fmt(fmt, FMT_SIZE, "%s", justify, width),
+                   *(const bool *)value ?  "true" : "false");
                break;
        case emitter_type_int:
-               EMIT_SIMPLE(int, "d")
+               EMIT_SIMPLE(int, "%d")
                break;
        case emitter_type_unsigned:
-               EMIT_SIMPLE(unsigned, "u")
+               EMIT_SIMPLE(unsigned, "%u")
                break;
        case emitter_type_ssize:
-               EMIT_SIMPLE(ssize_t, "zd")
+               EMIT_SIMPLE(ssize_t, "%zd")
                break;
        case emitter_type_size:
-               EMIT_SIMPLE(size_t, "zu")
+               EMIT_SIMPLE(size_t, "%zu")
                break;
        case emitter_type_string:
                str_written = malloc_snprintf(buf, BUF_SIZE, "\"%s\"",
@@ -185,17 +188,17 @@
                 * anywhere near the fmt size.
                 */
                assert(str_written < BUF_SIZE);
-               emitter_gen_fmt(fmt, FMT_SIZE, "s", justify, width);
-               emitter_printf(emitter, fmt, buf);
+               emitter_printf(emitter, 
+                   emitter_gen_fmt(fmt, FMT_SIZE, "%s", justify, width), buf);
                break;
        case emitter_type_uint32:
-               EMIT_SIMPLE(uint32_t, FMTu32)
+               EMIT_SIMPLE(uint32_t, "%" FMTu32)
                break;
        case emitter_type_uint64:
-               EMIT_SIMPLE(uint64_t, FMTu64)
+               EMIT_SIMPLE(uint64_t, "%" FMTu64)
                break;
        case emitter_type_title:
-               EMIT_SIMPLE(char *const, "s");
+               EMIT_SIMPLE(char *const, "%s");
                break;
        default:
                unreachable();



Home | Main Index | Thread Index | Old Index