summaryrefslogtreecommitdiff
path: root/diff.c
diff options
context:
space:
mode:
authorJunio C Hamano <gitster@pobox.com>2024-11-25 12:29:39 +0900
committerJunio C Hamano <gitster@pobox.com>2024-11-25 12:29:39 +0900
commit6b03fd8dcd1fd88ad834a3b3f678631d6d68ea54 (patch)
treef952c57b48b9fad56e97b3026b9c148a2a3dcb2e /diff.c
parent304e77d2f8fc6474281c0b767f5a918d036a0780 (diff)
parent1164e270b5af80516625b628945ec7365d992055 (diff)
Merge branch 'jk/output-prefix-cleanup' into maint-2.47
Code clean-up. * jk/output-prefix-cleanup: diff: store graph prefix buf in git_graph struct diff: return line_prefix directly when possible diff: return const char from output_prefix callback diff: drop line_prefix_length field line-log: use diff_line_prefix() instead of custom helper
Diffstat (limited to 'diff.c')
-rw-r--r--diff.c10
1 files changed, 3 insertions, 7 deletions
diff --git a/diff.c b/diff.c
index 173cbe2bed..6c96154fed 100644
--- a/diff.c
+++ b/diff.c
@@ -2317,12 +2317,9 @@ const char *diff_get_color(int diff_use_color, enum color_diff ix)
const char *diff_line_prefix(struct diff_options *opt)
{
- struct strbuf *msgbuf;
- if (!opt->output_prefix)
- return "";
-
- msgbuf = opt->output_prefix(opt, opt->output_prefix_data);
- return msgbuf->buf;
+ return opt->output_prefix ?
+ opt->output_prefix(opt, opt->output_prefix_data) :
+ "";
}
static unsigned long sane_truncate_line(char *line, unsigned long len)
@@ -5400,7 +5397,6 @@ static int diff_opt_line_prefix(const struct option *opt,
BUG_ON_OPT_NEG(unset);
options->line_prefix = optarg;
- options->line_prefix_length = strlen(options->line_prefix);
graph_setup_line_prefix(options);
return 0;
}