diff options
author | Jeff King <peff@peff.net> | 2013-07-10 02:18:40 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2013-07-10 07:23:04 +0200 |
commit | 4621085b7eb2f4cffe16d508988ff9b4a874b4ef (patch) | |
tree | e00d04f064c23918768aafaa93656a07949a95db /trace.c | |
parent | Git 1.8.3 (diff) | |
download | git-4621085b7eb2f4cffe16d508988ff9b4a874b4ef.tar.xz git-4621085b7eb2f4cffe16d508988ff9b4a874b4ef.zip |
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 'trace.c')
-rw-r--r-- | trace.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -75,6 +75,7 @@ static void trace_vprintf(const char *key, const char *fmt, va_list ap) strbuf_release(&buf); } +__attribute__((format (printf, 2, 3))) static void trace_printf_key(const char *key, const char *fmt, ...) { va_list ap; |