Fix type of combine-diff.c::show_patch_diff()
The other function, show_raw_diff() is void and no callers use return value from neither. Signed-off-by: Junio C Hamano <junkio@cox.net>
This commit is contained in:
parent
182a8dabd5
commit
89b0c4b5a3
@ -609,7 +609,7 @@ static void dump_quoted_path(const char *prefix, const char *path,
|
||||
printf("%s\n", c_reset);
|
||||
}
|
||||
|
||||
static int show_patch_diff(struct combine_diff_path *elem, int num_parent,
|
||||
static void show_patch_diff(struct combine_diff_path *elem, int num_parent,
|
||||
int dense, struct rev_info *rev)
|
||||
{
|
||||
struct diff_options *opt = &rev->diffopt;
|
||||
@ -617,7 +617,7 @@ static int show_patch_diff(struct combine_diff_path *elem, int num_parent,
|
||||
char *result, *cp;
|
||||
struct sline *sline; /* survived lines */
|
||||
int mode_differs = 0;
|
||||
int i, show_hunks, shown_header = 0;
|
||||
int i, show_hunks;
|
||||
int working_tree_file = !memcmp(elem->sha1, null_sha1, 20);
|
||||
int abbrev = opt->full_index ? 40 : DEFAULT_ABBREV;
|
||||
mmfile_t result_file;
|
||||
@ -769,7 +769,6 @@ static int show_patch_diff(struct combine_diff_path *elem, int num_parent,
|
||||
}
|
||||
free(sline[0].p_lno);
|
||||
free(sline);
|
||||
return shown_header;
|
||||
}
|
||||
|
||||
#define COLONS "::::::::::::::::::::::::::::::::"
|
||||
@ -837,12 +836,11 @@ void show_combined_diff(struct combine_diff_path *p,
|
||||
return;
|
||||
if (opt->output_format & (DIFF_FORMAT_RAW |
|
||||
DIFF_FORMAT_NAME |
|
||||
DIFF_FORMAT_NAME_STATUS)) {
|
||||
DIFF_FORMAT_NAME_STATUS))
|
||||
show_raw_diff(p, num_parent, rev);
|
||||
} else if (opt->output_format & DIFF_FORMAT_PATCH) {
|
||||
else if (opt->output_format & DIFF_FORMAT_PATCH)
|
||||
show_patch_diff(p, num_parent, dense, rev);
|
||||
}
|
||||
}
|
||||
|
||||
void diff_tree_combined(const unsigned char *sha1,
|
||||
const unsigned char parent[][20],
|
||||
|
Loading…
Reference in New Issue
Block a user