"Nick Edelen" <sirnot@xxxxxxxxx> writes:
> This patch provides a working integration of rev-cache into the revision
> walker, along with some touch-ups:
This message from me is not about the patch 5/6 at all. It is about your
earlier patches in the series.
> ...
> - printf("queue:\n");
> + fprintf(stderr, "queue:\n");
> ...
> - printf("work:\n");
> + fprintf(stderr, "work:\n");
> ...
> - printf("pending:\n");
> + fprintf(stderr, "pending:\n");
> ...
> - pptr = &item->parents;
> + pptr = &item->parents;
> ...
> -#define SLOP 5
> +#define SLOP 5
> ...
These are all "Shoot, the mistakes in earlier ones need to be fixed here"
changes.
Please fix them in the patch earlier in the series so that you do not have
to make, and more importantly, reviewers do not have to get distracted by,
these fix-up in a later patch like this.
--
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