> 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. > +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?