Stefan Beller <sbeller@xxxxxxxxxx> writes: > Additionally each patch adds a semantic patch, that would port from the old to > the new function. These semantic patches are all applied in the very last patch, > but we could omit applying the last patch if it causes too many merge conflicts > and trickl in the semantic patches over time when there are no merge conflicts. That's an interesting approach ;-) > The original goal of all these refactoring series was to remove add_submodule_odb > in submodule.c, which was partially reached with this series. Yup, that is a very good goalpost to keep in mind. > remaining calls in another series, but it shows we're close to be done with these > large refactorings as far as I am concerned. Nice.