Mike Rappazzo <rappazzo@xxxxxxxxx> writes: > On Fri, Sep 11, 2015 at 12:16 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: >> Michael Rappazzo <rappazzo@xxxxxxxxx> writes: >> >>> The code formerly in branch.c was largely the basis of the >>> get_worktree_list implementation is now moved to worktree.c, >>> and the find_shared_symref implementation has been refactored >>> to use get_worktree_list >>> >> >> Copying the bulk of the function in 1/3 and then removing the >> original here made it somewhat hard to compare what got changed in >> the implementation. >> >> I _think_ the code structure in the end result is more or less >> right, though. > > Should I squash these first two commits together in the next series? Mashing the two into one patch would not help anybody, I would suspect. I didn't try this myself, but if I were doing this and if I were aiming for perfection, then I would probably try to split it even finer. Refactor the original while both the callers and the helpers are still inside branch.c (hence the early steps in the series will not benefit worktree.c at all---worktree.c may not even exist in them yet), move refactored helpers to worktree.[ch] (and at this step you may not even have get_worktree() etc. yet), and then use that refactored helper while writing new functions in worktree.c. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html