summaryrefslogtreecommitdiff
path: root/src/patch_parse.c
diff options
context:
space:
mode:
authorPatrick Steinhardt <ps@pks.im>2017-07-14 15:33:32 +0200
committerPatrick Steinhardt <ps@pks.im>2017-11-11 17:06:23 +0000
commit7bdfc0a68506709b05c40ac519ded044a5ad7124 (patch)
treeff17093c4ec6b03ee3b14f5c8af7691e6bc42bcb /src/patch_parse.c
parente72cb769f6b8c11344463a9784fd4d32ded4a685 (diff)
downloadlibgit2-7bdfc0a68506709b05c40ac519ded044a5ad7124.tar.gz
parse: always initialize line pointer
Upon initializing the parser context, we do not currently initialize the current line, line length and line number. Do so in order to make the interface easier to use and more obvious for future consumers of the parsing API.
Diffstat (limited to 'src/patch_parse.c')
-rw-r--r--src/patch_parse.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/src/patch_parse.c b/src/patch_parse.c
index 8b0a5d351..fee6afad6 100644
--- a/src/patch_parse.c
+++ b/src/patch_parse.c
@@ -613,10 +613,7 @@ static int parse_patch_header(
{
int error = 0;
- for (ctx->parse_ctx.line = ctx->parse_ctx.remain;
- ctx->parse_ctx.remain_len > 0;
- git_parse_advance_line(&ctx->parse_ctx)) {
-
+ for (; ctx->parse_ctx.remain_len > 0; git_parse_advance_line(&ctx->parse_ctx)) {
/* This line is too short to be a patch header. */
if (ctx->parse_ctx.line_len < 6)
continue;