On Wed, Aug 05, 2009 at 02:15:44AM -0700, Junio C Hamano wrote: > 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: > 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 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html