Junio C Hamano wrote: > Jonathan Nieder <jrnieder@xxxxxxxxx> writes: > >> @@ -209,33 +236,11 @@ static int list_prerequisites(int bundle_fd, struct rev_info *revs, >> if (prepare_revision_walk(&boundary_revs)) >> return error("revision walk setup failed"); >> >> - while ((rev = get_revision(&boundary_revs))) { >> - if (rev->object.flags & BOUNDARY) { >> -... >> - } else { >> + while ((rev = get_revision(revs))) { >> + if (rev->object.flags & BOUNDARY) >> + list_prerequisite(bundle_fd, revs, rev); >> + else >> rev->object.flags |= SHOWN; >> - } > > You used to walk boundary_revs but now you walk revs that is given by the > caller Agh! Typo. Thanks for catching it. -- 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