diff options
author | Junio C Hamano <gitster@pobox.com> | 2024-04-12 11:31:38 -0700 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2024-04-12 11:31:38 -0700 |
commit | d8800f630a55264c11693d665846c96caa5a4d1c (patch) | |
tree | 712caeefd7b1998b0d6c5d9d1147135d2e53dece | |
parent | d842e22ebbb2d70e11f8055c5cd7d1d4e1a90e2e (diff) | |
parent | 39bb692152ab8ccd3f955548ad32182c32af529a (diff) | |
download | git-d8800f630a55264c11693d665846c96caa5a4d1c.tar.gz |
Merge branch 'rs/imap-send-use-xsnprintf'
Code clean-up and duplicate reduction.
* rs/imap-send-use-xsnprintf:
imap-send: use xsnprintf to format command
-rw-r--r-- | imap-send.c | 24 |
1 files changed, 4 insertions, 20 deletions
diff --git a/imap-send.c b/imap-send.c index f2e1947e63..4caa8668e6 100644 --- a/imap-send.c +++ b/imap-send.c @@ -68,9 +68,6 @@ static void imap_warn(const char *, ...); static char *next_arg(char **); -__attribute__((format (printf, 3, 4))) -static int nfsnprintf(char *buf, int blen, const char *fmt, ...); - static int nfvasprintf(char **strp, const char *fmt, va_list ap) { int len; @@ -500,19 +497,6 @@ static char *next_arg(char **s) return ret; } -__attribute__((format (printf, 3, 4))) -static int nfsnprintf(char *buf, int blen, const char *fmt, ...) -{ - int ret; - va_list va; - - va_start(va, fmt); - if (blen <= 0 || (unsigned)(ret = vsnprintf(buf, blen, fmt, va)) >= (unsigned)blen) - BUG("buffer too small. Please report a bug."); - va_end(va); - return ret; -} - static struct imap_cmd *issue_imap_cmd(struct imap_store *ctx, struct imap_cmd_cb *cb, const char *fmt, va_list ap) @@ -535,11 +519,11 @@ static struct imap_cmd *issue_imap_cmd(struct imap_store *ctx, get_cmd_result(ctx, NULL); if (!cmd->cb.data) - bufl = nfsnprintf(buf, sizeof(buf), "%d %s\r\n", cmd->tag, cmd->cmd); + bufl = xsnprintf(buf, sizeof(buf), "%d %s\r\n", cmd->tag, cmd->cmd); else - bufl = nfsnprintf(buf, sizeof(buf), "%d %s{%d%s}\r\n", - cmd->tag, cmd->cmd, cmd->cb.dlen, - CAP(LITERALPLUS) ? "+" : ""); + bufl = xsnprintf(buf, sizeof(buf), "%d %s{%d%s}\r\n", + cmd->tag, cmd->cmd, cmd->cb.dlen, + CAP(LITERALPLUS) ? "+" : ""); if (0 < verbosity) { if (imap->num_in_progress) |