merge-recursive: Small code clarification -- variable name and comments

process_renames() had a variable named "stage" and derived variables
src_other and dst_other whose purpose was not immediately obvious; also,
I want to extend the scope of this variable and use it later, so it should
have a more descriptive name.  Do so, and add a brief comment explaining
how it is used and what it relates to.

Signed-off-by: Elijah Newren <newren@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
Elijah Newren 2010-09-20 02:28:47 -06:00 коммит произвёл Junio C Hamano
Родитель f63622c0a9
Коммит 41d70bd6a9
1 изменённых файлов: 14 добавлений и 6 удалений

Просмотреть файл

@ -924,15 +924,23 @@ static int process_renames(struct merge_options *o,
struct string_list_item *item; struct string_list_item *item;
/* we only use sha1 and mode of these */ /* we only use sha1 and mode of these */
struct diff_filespec src_other, dst_other; struct diff_filespec src_other, dst_other;
int try_merge, stage = a_renames == renames1 ? 3: 2; int try_merge;
remove_file(o, 1, ren1_src, o->call_depth || stage == 3); /*
* unpack_trees loads entries from common-commit
* into stage 1, from head-commit into stage 2, and
* from merge-commit into stage 3. We keep track
* of which side corresponds to the rename.
*/
int renamed_stage = a_renames == renames1 ? 2 : 3;
int other_stage = a_renames == renames1 ? 3 : 2;
hashcpy(src_other.sha1, ren1->src_entry->stages[stage].sha); remove_file(o, 1, ren1_src, o->call_depth || renamed_stage == 2);
src_other.mode = ren1->src_entry->stages[stage].mode;
hashcpy(dst_other.sha1, ren1->dst_entry->stages[stage].sha);
dst_other.mode = ren1->dst_entry->stages[stage].mode;
hashcpy(src_other.sha1, ren1->src_entry->stages[other_stage].sha);
src_other.mode = ren1->src_entry->stages[other_stage].mode;
hashcpy(dst_other.sha1, ren1->dst_entry->stages[other_stage].sha);
dst_other.mode = ren1->dst_entry->stages[other_stage].mode;
try_merge = 0; try_merge = 0;
if (sha_eq(src_other.sha1, null_sha1)) { if (sha_eq(src_other.sha1, null_sha1)) {