]> git.ipfire.org Git - thirdparty/git.git/commit - remote-curl.c
C: use skip_prefix() to avoid hardcoded string length
authorJunio C Hamano <gitster@pobox.com>
Thu, 30 Jan 2020 19:35:46 +0000 (11:35 -0800)
committerJunio C Hamano <gitster@pobox.com>
Fri, 31 Jan 2020 21:03:45 +0000 (13:03 -0800)
commit145136a95a8755528aa012a4ce0ed50d1ec39e24
treecdb54dc3d1730adf84bff93e743f55414ea8ea0e
parentd0654dc308b0ba76dd8ed7bbb33c8d8f7aacd783
C: use skip_prefix() to avoid hardcoded string length

We often skip an optional prefix in a string with a hardcoded
constant, e.g.

if (starts_with(string, "prefix"))
string += 6;

which is less error prone when written

skip_prefix(string, "prefix", &string);

Note that this changes a few error messages from "git reflog expire
--expire=nonsense.timestamp", which used to complain by saying

    '--expire=nonsense.timestamp' is not a valid timestamp

but with this change, we say

    'nonsense.timestamp' is not a valid timestamp

which is more technically correct (the string with --expire= as
a prefix obviously cannot be a valid timestamp, but the error is
about the part of the input without that prefix).

Helped-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/fast-export.c
builtin/reflog.c
notes.c
parse-options.c
refs/files-backend.c
remote-curl.c
sha1-name.c
transport-helper.c