Re: [PATCH v5 2/5] ref-cache: remove unused function 'find_ref_entry()'

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

 



Junio C Hamano <gitster@xxxxxxxxx> writes:

> Karthik Nayak <karthik.188@xxxxxxxxx> writes:
>
>> The 'find_ref_entry' function is no longer used, so remove it.
>
> If my spelunking is correct, ba1c052f (ref_store: implement
> `refs_peel_ref()` generically, 2017-09-25) is the commit that
> removed the last caller of it.  Which may be worth noting here.
>

Indeed, I could also verify this by running

$ git log -S find_ref_entry --oneline
2c90b85801 ref-cache: remove unused function 'find_ref_entry()'
ba1c052fa6 ref_store: implement `refs_peel_ref()` generically
9939b33d6a packed-backend: rip out some now-unused code
....

And looking at `ba1c052fa6`. I should've done this before. But thanks
for the digging!

I plan to address a few comments on this version, but I also see that
you've merged it to master. Should I raise followups for noteworthy
changes or a new version?

Karthik

Attachment: signature.asc
Description: PGP signature


[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