diff options
| author | Jeff King <peff@peff.net> | 2014-06-18 15:42:14 -0400 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2014-06-18 14:56:54 -0700 |
| commit | c0264180d74510669bcec9bb5ff12a145c685b8e (patch) | |
| tree | ecd1875c4e0e09af61b0b2321e4f1e6f851ed04d /builtin/fmt-merge-msg.c | |
| parent | 1055a890f04784d7ea7cdaca8d30479e685d8643 (diff) | |
| download | git-c0264180d74510669bcec9bb5ff12a145c685b8e.tar.gz | |
avoid using skip_prefix as a boolean
There's no point in using:
if (skip_prefix(buf, "foo"))
over
if (starts_with(buf, "foo"))
as the point of skip_prefix is to return a pointer to the
data after the prefix. Using starts_with is more readable,
and will make refactoring skip_prefix easier.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/fmt-merge-msg.c')
| -rw-r--r-- | builtin/fmt-merge-msg.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin/fmt-merge-msg.c b/builtin/fmt-merge-msg.c index 3906eda877..72378e6810 100644 --- a/builtin/fmt-merge-msg.c +++ b/builtin/fmt-merge-msg.c @@ -298,7 +298,7 @@ static void credit_people(struct strbuf *out, (them->nr == 1 && me && (me = skip_prefix(me, them->items->string)) != NULL && - skip_prefix(me, " <"))) + starts_with(me, " <"))) return; strbuf_addf(out, "\n%c %s ", comment_line_char, label); add_people_count(out, them); |