On Wed, Feb 16 2022, Glen Choo wrote: > struct string_list_item *item; > - for_each_string_list_item(item, submodules) > - oid_array_clear((struct oid_array *) item->util); > + for_each_string_list_item(item, submodules) { > + changed_submodule_data_clear(item->util); > + } Nit: These {} additions aren't needed here or in the end state of this series.