diff options
author | 2020-04-10 15:46:07 -0400 | |
---|---|---|
committer | 2020-04-10 14:52:22 -0700 | |
commit | f011a9654dc2a2e3238985ba7767c1058e7cb3c2 (patch) | |
tree | fd9be67e6b3e3d956b5f1264fa2719075780cb03 /INSTALL | |
parent | config: drop useless length variable in write_pair() (diff) | |
download | tgif-f011a9654dc2a2e3238985ba7767c1058e7cb3c2.tar.xz |
git_config_parse_key(): return baselen as size_t
As with the recent change to parse_config_key(), the best type to return
a string length is a size_t, as it won't cause integer truncation for a
gigantic key. And as with that change, this is mostly a clarity /
hygiene issue for now, as our config parser would choke on such a large
key anyway.
There are a few ripple effects within the config code, as callers switch
to using size_t. I also adjusted a few related variables that iterate
over strings. The most unexpected change is that a call to strbuf_addf()
had to switch to strbuf_add(). We can't use a size_t with "%.*s",
because printf precisions must have type "int" (we could cast, of
course, but that would miss the point of using size_t in the first
place).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'INSTALL')
0 files changed, 0 insertions, 0 deletions