login
Header Space

 
 

Re: [PATCH] name-rev: Fix non-shortest description

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Jeff King <peff@...>
Cc: Uwe <ukleinek@...>, <git@...>, <gitster@...>
Date: Tuesday, August 28, 2007 - 7:28 am

Hi,

On Tue, 28 Aug 2007, Jeff King wrote:


Good!


Well, we could actually do the correct thing, and not have a fifo, but a 
Fibonacci heap, so that we traverse the commits by cost.  Then there 
cannot be any clashes by definition.  First wins.


But of course!


Most comparisons (I guess) would be relatively cheap, since the traversals 
alone would suffice, and most of the others would be relatively short.

But I really wonder now if it is not just easier to increase 
MERGE_TRAVERSAL_WEIGHT to (1<<24) and be done with it once and (probably) 
for all.

Ciao,
Dscho

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
name-rev does not show the shortest path, Uwe , (Thu Aug 23, 6:38 am)
[PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Mon Aug 27, 7:37 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 4:20 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 5:15 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 5:25 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 6:03 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 6:26 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 7:02 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 7:11 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 7:28 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 7:34 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 4:39 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 5:00 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 5:26 am)
Re: name-rev does not show the shortest path, Julian Phillips, (Fri Aug 24, 7:55 am)
Re: name-rev does not show the shortest path, Julian Phillips, (Fri Aug 24, 11:21 am)
Re: name-rev does not show the shortest path, Junio C Hamano, (Fri Aug 24, 2:33 pm)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Sat Aug 25, 11:04 am)
Re: name-rev does not show the shortest path, Jeff King, (Sun Aug 26, 5:23 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Sun Aug 26, 11:38 am)
Re: name-rev does not show the shortest path, Jeff King, (Mon Aug 27, 5:24 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Mon Aug 27, 5:57 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Mon Aug 27, 7:18 am)
speck-geostationary