Home » Mailing list archives » git » 2007 » August » 28

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

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Johannes Schindelin
Subject: Re: [PATCH] name-rev: Fix non-shortest description
Date: Tuesday, August 28, 2007 - 2:15 am

Hi,

On Tue, 28 Aug 2007, Jeff King wrote:


This is exactly what I tried to avoid, and why I did _not_ do the 
"correct" fix.  In some real-life (i.e. non-OpenSource) repositories you 
do get a mess, and you do get quite a lot of messy merges.  So I am not at 
all convinced that this holds together in such setups.

Besides, name-rev is already a memory hog.  Your patch makes it worse.

IOW I think that my patch is a good trade off between correct and working.

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, 3:38 am)
Re: name-rev does not show the shortest path, Julian Phillips, (Fri Aug 24, 4:55 am)
Re: name-rev does not show the shortest path, Uwe , (Fri Aug 24, 5:52 am)
Re: name-rev does not show the shortest path, Julian Phillips, (Fri Aug 24, 8:21 am)
Re: name-rev does not show the shortest path, Junio C Hamano, (Fri Aug 24, 11:33 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Sat Aug 25, 8:04 am)
Re: name-rev does not show the shortest path, Jeff King, (Sun Aug 26, 2:23 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Sun Aug 26, 8:38 am)
Re: name-rev does not show the shortest path, Jeff King, (Mon Aug 27, 2:24 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Mon Aug 27, 2:57 am)
Re: name-rev does not show the shortest path, Johannes Schindelin, (Mon Aug 27, 4:18 am)
[PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Mon Aug 27, 4:37 am)
Re: [PATCH] name-rev: Fix non-shortest description, Uwe , (Mon Aug 27, 12:27 pm)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 1:20 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 1:39 am)
Re: [PATCH] name-rev: Fix non-shortest description, Uwe , (Tue Aug 28, 1:54 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 2:00 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 2:15 am)
Re: [PATCH] name-rev: Fix non-shortest description, Uwe , (Tue Aug 28, 2:17 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 2:25 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 2:26 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 3:03 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 3:26 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 4:02 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 4:11 am)
Re: [PATCH] name-rev: Fix non-shortest description, Johannes Schindelin, (Tue Aug 28, 4:28 am)
Re: [PATCH] name-rev: Fix non-shortest description, Jeff King, (Tue Aug 28, 4:34 am)