From 2011bb4f34d773a7de2d64769ca9f508feba8089 Mon Sep 17 00:00:00 2001 From: Jeff King Date: Thu, 3 Oct 2024 17:06:54 -0400 Subject: diff: drop line_prefix_length field The diff_options structure holds a line_prefix string and an associated length. But the length is always just the strlen() of the NUL-terminated string. Let's simplify the code by just storing the string pointer and assuming it is NUL-terminated when we use it. This will cause us to compute the string length in a few extra spots, but I don't think any of these are particularly hot code paths. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano --- graph.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) (limited to 'graph.c') diff --git a/graph.c b/graph.c index 1ca34770ee..34b18a80d4 100644 --- a/graph.c +++ b/graph.c @@ -74,10 +74,7 @@ static void graph_show_line_prefix(const struct diff_options *diffopt) if (!diffopt || !diffopt->line_prefix) return; - fwrite(diffopt->line_prefix, - sizeof(char), - diffopt->line_prefix_length, - diffopt->file); + fputs(diffopt->line_prefix, diffopt->file); } static const char **column_colors; @@ -321,8 +318,7 @@ static struct strbuf *diff_output_prefix_callback(struct diff_options *opt, void strbuf_reset(&msgbuf); if (opt->line_prefix) - strbuf_add(&msgbuf, opt->line_prefix, - opt->line_prefix_length); + strbuf_addstr(&msgbuf, opt->line_prefix); if (graph) graph_padding_line(graph, &msgbuf); return &msgbuf; -- cgit v1.3