Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH 5/6 (v2)] full integration of rev-cache into git's revision walker, completed test suite

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:47:12

"Nick Edelen" [off-list ref] 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.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help