Re: [PATCH v2 1/4] promisor-remote: read partialClone config here

> > Hmm...is your concern about the case in which
> > repository_format_partial_clone is uninitialized, or about ignoring a
> > potential NULL value? If the former, I don't see how your suggestion
> > fixes things, since extensions.partialclone may never have been in the
> > config in the first place (and would thus leave
> > repository_format_partial_clone uninitialized, if it weren't for the
> > fact that it is in static storage and thus initialized to 0). If the
> > latter, I guess I should be more detailed about how it's being handled
> > in setup.c (or maybe just leave out the comment altogether - the code
> > here can handle a NULL repository_format_partial_clone for some reason).
> 
> My comment was about the latter; I was trying to understand what the
> comment meant relative to that case, and how and where that case would
> be handled in the code.  With that frame of reference, the comment
> seemed misleading to me...though perhaps the comment was intended to
> answer some other question entirely.