On Sun, Apr 7, 2013 at 1:50 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Felipe Contreras <felipe.contreras@xxxxxxxxx> writes: > >> If we do it after the revision traversal we can be sure that this is >> indeed a commit that will be processed (i.e. not a merge) and it's the >> top most one (thus removing the NEEDSWORK comment). > > There may not be a single top-most one anyway (which is what that > "randomly pick" comment refers to and punts), so taking the tip > after traversal is just as good as doing it before. So this is a > good change, but it still is punting. Yeah, I thought of looking at 'git diff' and do the same thing, but then I realized 'git diff --stat' has the same issue, so I don't think it really matters. -- Felipe Contreras -- 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