diff options
author | Jeff King <peff@peff.net> | 2017-09-13 13:17:57 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2017-09-14 15:18:00 +0900 |
commit | d9bd4cbb9cce9f872cc4427d1c27a62c6768b12a (patch) | |
tree | 019bcc8e3e8d5a4d3e97061ecb2441e6710dc632 /t/t7500/add-content-and-comment | |
parent | notes-merge: use ssize_t for write_in_full() return value (diff) | |
download | tgif-d9bd4cbb9cce9f872cc4427d1c27a62c6768b12a.tar.xz |
config: flip return value of store_write_*()
The store_write_section() and store_write_pairs() functions
are basically high-level wrappers around write(). But their
return values are flipped from our usual convention, using
"1" for success and "0" for failure.
Let's flip them to follow the usual write() conventions and
update all callers. As these are local to config.c, it's
unlikely that we'd have new callers in any topics in flight
(which would be silently broken by our change). But just to
be on the safe side, let's rename them to just
write_section() and write_pairs(). That also accentuates
their relationship with write().
Signed-off-by: Jeff King <peff@peff.net>
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t7500/add-content-and-comment')
0 files changed, 0 insertions, 0 deletions