in vswprintf, initialize the FILE rather than memset-and-assign
authorRich Felker <dalias@aerifal.cx>
Sat, 24 Feb 2018 17:08:30 +0000 (12:08 -0500)
committerRich Felker <dalias@aerifal.cx>
Sat, 24 Feb 2018 17:08:30 +0000 (12:08 -0500)
this is the idiom that's used elsewhere and should be more efficient
or at least no worse.

src/stdio/vswprintf.c

index 6eb2f6acbb22c3e2f9e461df1220c0b38e976a4d..38efed65a7210a1a40fca765e81d14d0ae8126ee 100644 (file)
@@ -1,6 +1,5 @@
 #include "stdio_impl.h"
 #include <limits.h>
-#include <string.h>
 #include <errno.h>
 #include <stdint.h>
 #include <wchar.h>
@@ -37,17 +36,17 @@ static size_t sw_write(FILE *f, const unsigned char *s, size_t l)
 int vswprintf(wchar_t *restrict s, size_t n, const wchar_t *restrict fmt, va_list ap)
 {
        int r;
-       FILE f;
        unsigned char buf[256];
        struct cookie c = { s, n-1 };
+       FILE f = {
+               .lbf = EOF,
+               .write = sw_write,
+               .lock = -1,
+               .buf = buf,
+               .buf_size = sizeof buf,
+               .cookie = &c,
+       };
 
-       memset(&f, 0, sizeof(FILE));
-       f.lbf = EOF;
-       f.write = sw_write;
-       f.buf_size = sizeof buf;
-       f.buf = buf;
-       f.lock = -1;
-       f.cookie = &c;
        if (!n) {
                return -1;
        } else if (n > INT_MAX) {