diff options
author | Jeff King <peff@peff.net> | 2016-07-08 05:12:42 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2016-07-08 09:47:29 -0700 |
commit | e04d08a4b3373972717c805ae8e788219b873b2a (patch) | |
tree | d94a1eae9969ac1d71025b9c2a130f6a01827a31 /builtin/notes.c | |
parent | write_file: add pointer+len variant (diff) | |
download | tgif-e04d08a4b3373972717c805ae8e788219b873b2a.tar.xz |
write_file: add format attribute
This gives us compile-time checking of our format strings,
which is a good thing.
I had also hoped it would help with confusing write_file()
and write_file_buf(), since the former's "..." can make it
match the signature of the latter. But given that the buffer
for write_file_buf() is generally not a string literal, the
compiler won't complain unless -Wformat-nonliteral is on,
and that creates a ton of false positives elsewhere in the
code base.
While we're there, let's also give the function a docstring,
which it never had.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/notes.c')
0 files changed, 0 insertions, 0 deletions