summary refs log tree commit diff
path: root/utf8.h
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2013-07-09 20:18:40 -0400
committerJunio C Hamano <gitster@pobox.com>2013-07-09 22:23:04 -0700
commit4621085b7eb2f4cffe16d508988ff9b4a874b4ef (patch)
treee00d04f064c23918768aafaa93656a07949a95db /utf8.h
parentedca4152560522a431a51fc0a06147fc680b5b18 (diff)
add missing "format" function attributes
For most of our functions that take printf-like formats, we
use gcc's __attribute__((format)) to get compiler warnings
when the functions are misused. Let's give a few more
functions the same protection.

In most cases, the annotations do not uncover any actual
bugs; the only code change needed is that we passed a size_t
to transfer_debug, which expected an int. Since we expect
the passed-in value to be a relatively small buffer size
(and cast a similar value to int directly below), we can
just cast away the problem.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'utf8.h')
-rw-r--r--utf8.h1
1 files changed, 1 insertions, 0 deletions
diff --git a/utf8.h b/utf8.h
index 32a7bfb987..65d0e42b96 100644
--- a/utf8.h
+++ b/utf8.h
@@ -10,6 +10,7 @@ int utf8_strwidth(const char *string);
 int is_utf8(const char *text);
 int is_encoding_utf8(const char *name);
 int same_encoding(const char *, const char *);
+__attribute__((format (printf, 2, 3)))
 int utf8_fprintf(FILE *, const char *, ...);
 
 void strbuf_add_wrapped_text(struct strbuf *buf,