[PATCH 0/1] ls-refs.c: minimize number of refs visited · Jacob Vosmaer <hidden> · 2021-01-19
[PATCH 1/1] ls-refs.c: minimize number of refs visited · Jacob Vosmaer <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Taylor Blau <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Jacob Vosmaer <hidden> · 2021-01-19
[PATCH 0/2] ls-refs: only traverse through longest common ref prefix · Taylor Blau <hidden> · 2021-01-19
[PATCH 2/2] ls-refs.c: traverse longest common ref prefix · Taylor Blau <hidden> · 2021-01-19
Re: [PATCH 2/2] ls-refs.c: traverse longest common ref prefix · Jeff King <hidden> · 2021-01-19
Re: [PATCH 2/2] ls-refs.c: traverse longest common ref prefix · Taylor Blau <hidden> · 2021-01-19
Re: [PATCH 2/2] ls-refs.c: traverse longest common ref prefix · Jeff King <hidden> · 2021-01-20
Re: [PATCH 2/2] ls-refs.c: traverse longest common ref prefix · Jacob Vosmaer <hidden> · 2021-01-20
[PATCH 1/2] refs: expose 'for_each_fullref_in_prefixes' · Taylor Blau <hidden> · 2021-01-19
[PATCH v2 0/3] ls-refs: traverse prefixes of disjoint "ref-prefix" sets · Taylor Blau <hidden> · 2021-01-20
[PATCH v2 2/3] ls-refs.c: initialize 'prefixes' before using it · Taylor Blau <hidden> · 2021-01-20
Re: [PATCH v2 2/3] ls-refs.c: initialize 'prefixes' before using it · Jeff King <hidden> · 2021-01-20
Re: [PATCH v2 2/3] ls-refs.c: initialize 'prefixes' before using it · Taylor Blau <hidden> · 2021-01-20
Re: [PATCH v2 2/3] ls-refs.c: initialize 'prefixes' before using it · Jacob Vosmaer <hidden> · 2021-01-20
[PATCH v2 3/3] ls-refs.c: traverse prefixes of disjoint "ref-prefix" sets · Taylor Blau <hidden> · 2021-01-20
[PATCH v2 1/3] refs: expose 'for_each_fullref_in_prefixes' · Taylor Blau <hidden> · 2021-01-20
Re: [PATCH v2 1/3] refs: expose 'for_each_fullref_in_prefixes' · Jeff King <hidden> · 2021-01-20
Re: [PATCH v2 1/3] refs: expose 'for_each_fullref_in_prefixes' · Taylor Blau <hidden> · 2021-01-20
Re: [PATCH v2 0/3] ls-refs: traverse prefixes of disjoint "ref-prefix" sets · Junio C Hamano <hidden> · 2021-01-23
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Taylor Blau <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Jeff King <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Jeff King <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Taylor Blau <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Jeff King <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Jeff King <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Taylor Blau <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Jeff King <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Taylor Blau <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Jeff King <hidden> · 2021-01-19
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Jacob Vosmaer <hidden> · 2021-01-20
Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited · Jacob Vosmaer <hidden> · 2021-01-20

Re: [PATCH 1/1] ls-refs.c: minimize number of refs visited

From: Jacob Vosmaer <hidden>
Date: 2021-01-20 11:34:09

On Wed, Jan 20, 2021 at 11:40 AM Jacob Vosmaer [off-list ref] wrote:

I also think we could. But as I alluded to in my original commit
message, I don't like how complicated that gets. I find it easier to
convince myself in the current form that the longest prefix code
selects _enough_ prefixes, which is a weaker property than "selects
exactly the right prefixes".

By which I mean to say I think we should keep the current form, with
the final authoritative ref matching pass.

Jacob

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help