]> git.ipfire.org Git - thirdparty/git.git/commit - cache.h
write_file: add format attribute
authorJeff King <peff@peff.net>
Fri, 8 Jul 2016 09:12:42 +0000 (05:12 -0400)
committerJunio C Hamano <gitster@pobox.com>
Fri, 8 Jul 2016 16:47:29 +0000 (09:47 -0700)
commite04d08a4b3373972717c805ae8e788219b873b2a
treed94a1eae9969ac1d71025b9c2a130f6a01827a31
parent52563d7ecc8f3f38cb1c0521294c5f6a0a475637
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>
cache.h