"brian m. carlson" [off-list ref] writes:
quoted
+#include "strbuf.h"
+#include "sha1-array.h"
+
+struct alternates {
+ struct alternate_object_database *list;
+ struct alternate_object_database **tail;
+};
+#define ALTERNATES_INIT { NULL, NULL }
I was surprised to find that this patch not only moves the alternates
API to a new location, but introduces this struct. I certainly think
the struct is a good idea, but it should probably go in a separate
patch, or at least get a mention in the commit message.
Yeah, I tend to agree that splitting it into two patches may make
more sense, especially given that this is already close to 200 patch
series ;-)