Re: [PATCH 1/7] submodule: lazily add submodule ODBs as alternates

quoted

+static struct string_list added_submodule_odb_paths = STRING_LIST_INIT_NODUP;
+

I see 2/7 allows callers to add paths for submodule odb to this
list.

Yes, and as you said below, add_submodule_odb() has been modified to add
to this list too.

The "pretend as if objects in submodule odb are locally available",
even though it may have been useful, was an ugly hack invented
before we started adding the "access more than one repository at the
time with the 'repo' arg" extended API.  It is pleasing to see this
step shows an approach to incrementally migrate the users of the
hack.

Thanks.