Thread (1 message) 1 message, 1 author, 2021-01-22

Re: [PATCH] refs: switch peel_ref() to peel_iterated_oid()

From: Junio C Hamano <hidden>
Date: 2021-01-22 21:41:03

Taylor Blau [off-list ref] writes:
On Wed, Jan 20, 2021 at 02:44:43PM -0500, Jeff King wrote:
quoted
This has been bugging me for years, and came up again in:

  https://lore.kernel.org/git/YAhXw9Gvn5Pyvacq@coredump.intra.peff.net/ (local)

so I decided to finally do something about it. There's a minor textual
conflict with the caller being touched in that thread, but it's easy to
resolve. So I think it's OK to consider the two independent.

The commit message turned out rather long. I hope that doesn't scare
anybody away, but I wanted to cover all of the bases.
Thanks for all of the detail. Suffice it to say, I was not scared away,
and am in fact quite comfortable with this direction.

Sorry for sending such a short response to such a long email, but I
agree with the discussion in the patch message, and I don't have
anything to add other than my approval and a:

  Reviewed-by: Taylor Blau [off-list ref]

Thanks,
Taylor
Thanks, both.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help