On Mon, Oct 3, 2016 at 1:35 PM, Jeff King <peff@xxxxxxxx> wrote: > Allocating a struct alternate_object_database is tricky, as > we must over-allocate the buffer to provide scratch space, > and then put in particular '/' and NUL markers. > > Let's encapsulate this in a function so that the complexity > doesn't leak into callers (and so that we can modify it > later). The overall way this was broken up is definitely a lot of patches to follow but understanding the end goal this is a huge improvement in code maintainability here. This original allocation is indeed very tricky. Thanks, Jake