diff options
| author | Elijah Newren <newren@gmail.com> | 2018-02-14 10:51:50 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2018-02-14 13:02:52 -0800 |
| commit | 2dd6f8aa8495d30c9b16ed361c0b280bbed4bc14 (patch) | |
| tree | 56c5966f1590070dc09c42dd3f557a921a2b0bea /commit.c | |
| parent | merge-recursive: move the get_renames() function (diff) | |
| download | git-2dd6f8aa8495d30c9b16ed361c0b280bbed4bc14.tar.gz git-2dd6f8aa8495d30c9b16ed361c0b280bbed4bc14.zip | |
merge-recursive: introduce new functions to handle rename logic
The amount of logic in merge_trees() relative to renames was just a few
lines, but split it out into new handle_renames() and cleanup_renames()
functions to prepare for additional logic to be added to each. No code or
logic changes, just a new place to put stuff for when the rename detection
gains additional checks.
Note that process_renames() records pointers to various information (such
as diff_filepairs) into rename_conflict_info structs. Even though the
rename string_lists are not directly used once handle_renames() completes,
we should not immediately free the lists at the end of that function
because they store the information referenced in the rename_conflict_info,
which is used later in process_entry(). Thus the reason for a separate
cleanup_renames().
Reviewed-by: Stefan Beller <sbeller@google.com>
Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'commit.c')
0 files changed, 0 insertions, 0 deletions
