diff options
author | René Scharfe <l.s.r@web.de> | 2014-06-13 23:53:03 +0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-06-14 01:52:16 +0400 |
commit | 29aa0b2061712c83ec5eb5c1556436b1218035ba (patch) | |
tree | 5b6f8d48ec1f3cf414f1a68836629bd89e390a41 /builtin | |
parent | e156455ea49124c140a67623f22a393db62d5d98 (diff) |
blame: factor out get_next_line()
Move the code for finding the start of the next line into a helper
function in order to reduce duplication.
Signed-off-by: Rene Scharfe <l.s.r@web.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin')
-rw-r--r-- | builtin/blame.c | 28 |
1 files changed, 10 insertions, 18 deletions
diff --git a/builtin/blame.c b/builtin/blame.c index 88cb799727..f685b38a2f 100644 --- a/builtin/blame.c +++ b/builtin/blame.c @@ -1741,6 +1741,12 @@ static void output(struct scoreboard *sb, int option) } } +static const char *get_next_line(const char *start, const char *end) +{ + const char *nl = memchr(start, '\n', end - start); + return nl ? nl + 1 : NULL; +} + /* * To allow quick access to the contents of nth line in the * final image, prepare an index in the scoreboard. @@ -1754,15 +1760,8 @@ static int prepare_lines(struct scoreboard *sb) int *lineno; int num = 0, incomplete = 0; - for (p = buf;;) { - p = memchr(p, '\n', end - p); - if (p) { - p++; - num++; - continue; - } - break; - } + for (p = get_next_line(buf, end); p; p = get_next_line(p, end)) + num++; if (len && end[-1] != '\n') incomplete++; /* incomplete line at the end */ @@ -1771,15 +1770,8 @@ static int prepare_lines(struct scoreboard *sb) lineno = sb->lineno; *lineno++ = 0; - for (p = buf;;) { - p = memchr(p, '\n', end - p); - if (p) { - p++; - *lineno++ = p - buf; - continue; - } - break; - } + for (p = get_next_line(buf, end); p; p = get_next_line(p, end)) + *lineno++ = p - buf; if (incomplete) *lineno++ = len; |