On Tue, Jan 30, 2018 at 3:25 PM, Elijah Newren <newren@xxxxxxxxx> wrote: > In anticipation of more involved cleanup to come, make a helper function > for doing the cleanup at the end of handle_renames. Rename the already > existing cleanup_rename[s]() to final_cleanup_rename[s](), name the new > helper initial_cleanup_rename(), and leave the big comment in the code > about why we can't do all the cleanup at once. > > Signed-off-by: Elijah Newren <newren@xxxxxxxxx> This mostly touches new code of the series, so it feels a bit unnecessary. I don't feel strongly about it being a separate patch, as the patches up to now are easy to review. (and squashing may harm reviewability) But the code makes sense. Stefan
