On Wed, Oct 17, 2018 at 2:26 PM Jonathan Tan <jonathanta...@google.com> wrote: > > > The `changed_submodule_names` are only used for fetching, so let's make it > > part of the struct that is passed around for fetching submodules. > > Keep the titles of commit messages to 50 characters or under.
renamed > > > +static void calculate_changed_submodule_paths( > > + struct submodule_parallel_fetch *spf) > > Instead of taking the entire struct, could this just take the list of > changed submodule names instead? I think so, done.