Re: [PATCH 05/12] for_each_alternate_ref: replace transport code with for-each-ref

From
Junio C Hamano <hidden>
Date
2017-01-25 19:00:50
Jeff King [off-list ref] writes:
...
This patch omits the peeled information from
for_each_alternate_ref entirely, and leaves it up to the
caller whether they want to dig up the information.

Signed-off-by: Jeff King <redacted>
---
I also tried adding "%(*objectname)" to for-each-ref to just
grab the peeled information, but the peel implementation in
ref-filter is _really_ slow. It doesn't use the packed-ref
peel information, and it doesn't recognize duplicates (so in
the 80 million case, it really parses 80 million tags).
That's an interesting tangent that may want to be looked into.
 transport.c | 48 ++++++++++++++++++++++++++++++++++++++----------
 1 file changed, 38 insertions(+), 10 deletions(-)
The updated code is a more pleasant read.
Thread (1 message) 1 message, 1 author, last reply 2017-01-25
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help