Re: [PATCH 0/5] diff output_prefix cleanups

[Derrick Stolee <stolee@gmail.com>]

On 10/3/24 5:05 PM, Jeff King wrote:
 > Here's what I would> put on top (but I would suggest making it a separate 
branch, since yours
> is a fairly urgent fix and mine is all cleanup).
> 
>    [1/5]: line-log: use diff_line_prefix() instead of custom helper
>    [2/5]: diff: drop line_prefix_length field
>    [3/5]: diff: return const char from output_prefix callback
>    [4/5]: diff: return line_prefix directly when possible
>    [5/5]: diff: store graph prefix buf in git_graph struct

I've reviewed these patches and they look good to me. Thanks for
taking the time to split them up carefully to help review go so
smoothly.

>   7 files changed, 28 insertions(+), 43 deletions(-)

Excellent to see the line reduction, too.

Thanks,
-Stolee