Re: [PATCH 3/3] rev-list --bisect: Bisection "distance" clean up.

From: Junio C Hamano <gitster@pobox.com>
Date: Sep 16, 2007, 08:54 UTC
Message-ID: <7vfy1f56tt.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <20070915130016.eac885f4.chriscool@tuxfamily.org>

Christian Couder <chriscool@tuxfamily.org> writes:

Show 11 quoted lines

> diff --git a/builtin-rev-list.c b/builtin-rev-list.c
> index c2ce1fc..6ade3b7 100644
> --- a/builtin-rev-list.c
> +++ b/builtin-rev-list.c
> @@ -189,7 +189,7 @@ static int count_interesting_parents(struct commit *commit)
>  	return count;
>  }
>  
> -static inline int halfway(struct commit_list *p, int distance, int nr)
> +static inline int halfway(struct commit_list *p, int nr)
>  {

This makes sense as we always call with "distance = weight(p)".

But is this three-patch series really needed? We see this kind of clean-ups often as a prelude to a larger set of enhancements, building up anticipation for smarter and more beautiful things to come, and seeing the series end here leaves me wondering "Huh? Is that it?".

Not that clean-up itself does not have any value, but still...