Re: [PATCH 0/5] diff output_prefix cleanups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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





[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux