From fa270d92873a7fd595e093eb3c07409e336620c0 Mon Sep 17 00:00:00 2001 From: Curtis McEnroe Date: Tue, 7 Aug 2018 14:11:19 -0400 Subject: Hack clang into checking uiFmt format strings --- chat.h | 12 ++++++++++-- handle.c | 28 ++++++++++++++-------------- input.c | 2 +- irc.c | 4 ++-- ui.c | 1 + 5 files changed, 28 insertions(+), 19 deletions(-) diff --git a/chat.h b/chat.h index 3d3586c..47a7e1e 100644 --- a/chat.h +++ b/chat.h @@ -45,10 +45,18 @@ void uiRead(void); void uiTopic(const wchar_t *topic); void uiTopicStr(const char *topic); void uiLog(const wchar_t *line); - -//__attribute__((format(printf, 1, 2))) void uiFmt(const wchar_t *format, ...); +// HACK: clang won't check wchar_t *format strings. +#ifdef NDEBUG +#define uiFmt(format, ...) uiFmt(L##format, __VA_ARGS__) +#else +#define uiFmt(format, ...) do { \ + snprintf(NULL, 0, format, __VA_ARGS__); \ + uiFmt(L##format, __VA_ARGS__); \ +} while(0) +#endif + void handle(char *line); void input(wchar_t *line); diff --git a/handle.c b/handle.c index 9350e7f..23d3cba 100644 --- a/handle.c +++ b/handle.c @@ -59,7 +59,7 @@ static void handle432(char *prefix, char *params) { shift(¶ms); char *mesg = shift(¶ms); uiLog(L"You can't use that name here"); - uiFmt(L"Sheriff says, \"%s\"", mesg); + uiFmt("Sheriff says, \"%s\"", mesg); uiLog(L"Type /nick to choose a new one"); } @@ -82,7 +82,7 @@ static void handleJoin(char *prefix, char *params) { chat.user = strdup(user); } uiFmt( - L"\3%d%s\3 arrives in \3%d%s\3", + "\3%d%s\3 arrives in \3%d%s\3", color(user), nick, color(chan), chan ); } @@ -94,12 +94,12 @@ static void handlePart(char *prefix, char *params) { if (params) { char *mesg = shift(¶ms); uiFmt( - L"\3%d%s\3 leaves \3%d%s\3, \"%s\"", + "\3%d%s\3 leaves \3%d%s\3, \"%s\"", color(user), nick, color(chan), chan, mesg ); } else { uiFmt( - L"\3%d%s\3 leaves \3%d%s\3", + "\3%d%s\3 leaves \3%d%s\3", color(user), nick, color(chan), chan ); } @@ -112,11 +112,11 @@ static void handleQuit(char *prefix, char *params) { char *mesg = shift(¶ms); char *quot = (mesg[0] == '"') ? "" : "\""; uiFmt( - L"\3%d%s\3 leaves, %s%s%s", + "\3%d%s\3 leaves, %s%s%s", color(user), nick, quot, mesg, quot ); } else { - uiFmt(L"\3%d%s\3 leaves", color(user), nick); + uiFmt("\3%d%s\3 leaves", color(user), nick); } } @@ -127,7 +127,7 @@ static void handleKick(char *prefix, char *params) { char *kick = shift(¶ms); char *mesg = shift(¶ms); uiFmt( - L"\3%d%s\3 kicks \3%d%s\3 out of \3%d%s\3, \"%s\"", + "\3%d%s\3 kicks \3%d%s\3 out of \3%d%s\3, \"%s\"", color(user), nick, color(kick), kick, color(chan), chan, mesg ); } @@ -138,7 +138,7 @@ static void handle332(char *prefix, char *params) { char *chan = shift(¶ms); char *topic = shift(¶ms); uiFmt( - L"The sign in \3%d%s\3 reads, \"%s\"", + "The sign in \3%d%s\3 reads, \"%s\"", color(chan), chan, topic ); uiTopicStr(topic); @@ -150,7 +150,7 @@ static void handleTopic(char *prefix, char *params) { char *chan = shift(¶ms); char *topic = shift(¶ms); uiFmt( - L"\3%d%s\3 places a new sign in \3%d%s\3, \"%s\"", + "\3%d%s\3 places a new sign in \3%d%s\3, \"%s\"", color(user), nick, color(chan), chan, topic ); uiTopicStr(topic); @@ -191,7 +191,7 @@ static void handle315(char *prefix, char *params) { char *chan = shift(¶ms); who.len = 0; uiFmt( - L"In \3%d%s\3 are %s", + "In \3%d%s\3 are %s", color(chan), chan, who.buf ); } @@ -205,7 +205,7 @@ static void handleNick(char *prefix, char *params) { chat.nick = strdup(next); } uiFmt( - L"\3%d%s\3 is now known as \3%d%s\3", + "\3%d%s\3 is now known as \3%d%s\3", color(user), prev, color(user), next ); } @@ -217,9 +217,9 @@ static void handlePrivmsg(char *prefix, char *params) { char *mesg = shift(¶ms); if (mesg[0] == '\1') { strsep(&mesg, " "); - uiFmt(L"* \3%d%s\3 %s", color(user), nick, strsep(&mesg, "\1")); + uiFmt("* \3%d%s\3 %s", color(user), nick, strsep(&mesg, "\1")); } else { - uiFmt(L"<\3%d%s\3> %s", color(user), nick, mesg); + uiFmt("<\3%d%s\3> %s", color(user), nick, mesg); } } @@ -229,7 +229,7 @@ static void handleNotice(char *prefix, char *params) { char *chan = shift(¶ms); char *mesg = shift(¶ms); if (strcmp(chat.chan, chan)) return; - uiFmt(L"-\3%d%s\3- %s", color(user), nick, mesg); + uiFmt("-\3%d%s\3- %s", color(user), nick, mesg); } static const struct { diff --git a/input.c b/input.c index a53b0a2..0b653eb 100644 --- a/input.c +++ b/input.c @@ -98,5 +98,5 @@ void input(wchar_t *input) { COMMANDS[i].handler(input); return; } - uiFmt(L"/%ls isn't a recognized command", command); + uiFmt("/%ls isn't a recognized command", command); } diff --git a/irc.c b/irc.c index 1692a73..02a9f64 100644 --- a/irc.c +++ b/irc.c @@ -99,7 +99,7 @@ void ircFmt(const char *format, ...) { int len = vasprintf(&buf, format, ap); va_end(ap); if (!buf) err(EX_OSERR, "vasprintf"); - if (chat.verbose) uiFmt(L"<<< %.*s", len - 2, buf); + if (chat.verbose) uiFmt("<<< %.*s", len - 2, buf); ircWrite(buf, len); free(buf); } @@ -119,7 +119,7 @@ void ircRead(void) { char *crlf, *line = buf; while ((crlf = strnstr(line, "\r\n", &buf[len] - line))) { crlf[0] = '\0'; - if (chat.verbose) uiFmt(L">>> %s", line); + if (chat.verbose) uiFmt(">>> %s", line); handle(line); line = &crlf[2]; } diff --git a/ui.c b/ui.c index 5670851..972d824 100644 --- a/ui.c +++ b/ui.c @@ -28,6 +28,7 @@ #include #include "chat.h" +#undef uiFmt #define MIN(a, b) ((a) < (b) ? (a) : (b)) #define MAX(a, b) ((a) > (b) ? (a) : (b)) -- cgit 1.4.1