Skip to content

Commit

Permalink
apply: only pass required data to check_header_line
Browse files Browse the repository at this point in the history
Currently the 'check_header_line()' function takes 'struct
apply_state' as parameter, even though it only needs the linenr from
that struct.

This function is in the callchain of 'parse_git_header()', which we
want to make more generally useful in a subsequent commit.  To make
that happen we only want to pass in the required data to
'parse_git_header()', and not the whole 'struct apply_state', and thus
we want functions in the callchain of 'parse_git_header()' to only
take arguments they really need.

Signed-off-by: Thomas Gummerer <t.gummerer@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
tgummerer authored and gitster committed Jul 9, 2019
1 parent 85c3713 commit 570fe99
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions apply.c
Original file line number Diff line number Diff line change
Expand Up @@ -1302,15 +1302,15 @@ static char *git_header_name(int p_value,
}
}

static int check_header_line(struct apply_state *state, struct patch *patch)
static int check_header_line(int linenr, struct patch *patch)
{
int extensions = (patch->is_delete == 1) + (patch->is_new == 1) +
(patch->is_rename == 1) + (patch->is_copy == 1);
if (extensions > 1)
return error(_("inconsistent header lines %d and %d"),
patch->extension_linenr, state->linenr);
patch->extension_linenr, linenr);
if (extensions && !patch->extension_linenr)
patch->extension_linenr = state->linenr;
patch->extension_linenr = linenr;
return 0;
}

Expand Down Expand Up @@ -1380,7 +1380,7 @@ static int parse_git_header(struct apply_state *state,
res = p->fn(state, line + oplen, patch);
if (res < 0)
return -1;
if (check_header_line(state, patch))
if (check_header_line(state->linenr, patch))
return -1;
if (res > 0)
return offset;
Expand Down

0 comments on commit 570fe99

Please sign in to comment.