Re: [PATCH 3/5] wt-status.c: rework the way changes to the index and work tree are summarized
- From
Jeff King <peff@peff.net>
- Date
- Aug 6, 2009, 14:46 UTC
- Message-ID
- <20090806144638.GA1970@coredump.intra.peff.net>
- In-Reply-To
- <1249463746-21538-4-git-send-email-gitster@pobox.com>
On Wed, Aug 05, 2009 at 02:15:44AM -0700, Junio C Hamano wrote:
Show 10 quoted lines
> diff --git a/wt-status.c b/wt-status.c
> index 47735d8..1614352 100644
> --- a/wt-status.c
> +++ b/wt-status.c
> @@ -20,6 +20,7 @@ static char wt_status_colors[][COLOR_MAXLEN] = {
> GIT_COLOR_RED, /* WT_STATUS_CHANGED */
> GIT_COLOR_RED, /* WT_STATUS_UNTRACKED */
> GIT_COLOR_RED, /* WT_STATUS_NOBRANCH */
> + GIT_COLOR_YELLOW, /* WT_STATUS_UNMERGED */
> };Does this belong in 3/5? It looks like the WT_STATUS_UNMERGED symbol is not used at all until 4/5, which seems like the more logical place (since it deals explicitly with unmerged entries). Also, why does it start yellow here and then turn to red in the next patch?
And related:
Show 7 quoted lines
> diff --git a/wt-status.h b/wt-status.h > index 78add09..f80142f 100644 > --- a/wt-status.h > +++ b/wt-status.h > [...] > WT_STATUS_NOBRANCH, > + WT_STATUS_UNMERGED,
-Peff