merge-ort: record the reason that we want a rename for a directory
[git] / diffcore-rename.c
1 /*
2  *
3  * Copyright (C) 2005 Junio C Hamano
4  */
5 #include "cache.h"
6 #include "diff.h"
7 #include "diffcore.h"
8 #include "object-store.h"
9 #include "hashmap.h"
10 #include "progress.h"
11 #include "promisor-remote.h"
12 #include "strmap.h"
13
14 /* Table of rename/copy destinations */
15
16 static struct diff_rename_dst {
17         struct diff_filepair *p;
18         struct diff_filespec *filespec_to_free;
19         int is_rename; /* false -> just a create; true -> rename or copy */
20 } *rename_dst;
21 static int rename_dst_nr, rename_dst_alloc;
22 /* Mapping from break source pathname to break destination index */
23 static struct strintmap *break_idx = NULL;
24
25 static struct diff_rename_dst *locate_rename_dst(struct diff_filepair *p)
26 {
27         /* Lookup by p->ONE->path */
28         int idx = break_idx ? strintmap_get(break_idx, p->one->path) : -1;
29         return (idx == -1) ? NULL : &rename_dst[idx];
30 }
31
32 /*
33  * Returns 0 on success, -1 if we found a duplicate.
34  */
35 static int add_rename_dst(struct diff_filepair *p)
36 {
37         ALLOC_GROW(rename_dst, rename_dst_nr + 1, rename_dst_alloc);
38         rename_dst[rename_dst_nr].p = p;
39         rename_dst[rename_dst_nr].filespec_to_free = NULL;
40         rename_dst[rename_dst_nr].is_rename = 0;
41         rename_dst_nr++;
42         return 0;
43 }
44
45 /* Table of rename/copy src files */
46 static struct diff_rename_src {
47         struct diff_filepair *p;
48         unsigned short score; /* to remember the break score */
49 } *rename_src;
50 static int rename_src_nr, rename_src_alloc;
51
52 static void register_rename_src(struct diff_filepair *p)
53 {
54         if (p->broken_pair) {
55                 if (!break_idx) {
56                         break_idx = xmalloc(sizeof(*break_idx));
57                         strintmap_init(break_idx, -1);
58                 }
59                 strintmap_set(break_idx, p->one->path, rename_dst_nr);
60         }
61
62         ALLOC_GROW(rename_src, rename_src_nr + 1, rename_src_alloc);
63         rename_src[rename_src_nr].p = p;
64         rename_src[rename_src_nr].score = p->score;
65         rename_src_nr++;
66 }
67
68 static int basename_same(struct diff_filespec *src, struct diff_filespec *dst)
69 {
70         int src_len = strlen(src->path), dst_len = strlen(dst->path);
71         while (src_len && dst_len) {
72                 char c1 = src->path[--src_len];
73                 char c2 = dst->path[--dst_len];
74                 if (c1 != c2)
75                         return 0;
76                 if (c1 == '/')
77                         return 1;
78         }
79         return (!src_len || src->path[src_len - 1] == '/') &&
80                 (!dst_len || dst->path[dst_len - 1] == '/');
81 }
82
83 struct diff_score {
84         int src; /* index in rename_src */
85         int dst; /* index in rename_dst */
86         unsigned short score;
87         short name_score;
88 };
89
90 struct prefetch_options {
91         struct repository *repo;
92         int skip_unmodified;
93 };
94 static void prefetch(void *prefetch_options)
95 {
96         struct prefetch_options *options = prefetch_options;
97         int i;
98         struct oid_array to_fetch = OID_ARRAY_INIT;
99
100         for (i = 0; i < rename_dst_nr; i++) {
101                 if (rename_dst[i].p->renamed_pair)
102                         /*
103                          * The loop in diffcore_rename() will not need these
104                          * blobs, so skip prefetching.
105                          */
106                         continue; /* already found exact match */
107                 diff_add_if_missing(options->repo, &to_fetch,
108                                     rename_dst[i].p->two);
109         }
110         for (i = 0; i < rename_src_nr; i++) {
111                 if (options->skip_unmodified &&
112                     diff_unmodified_pair(rename_src[i].p))
113                         /*
114                          * The loop in diffcore_rename() will not need these
115                          * blobs, so skip prefetching.
116                          */
117                         continue;
118                 diff_add_if_missing(options->repo, &to_fetch,
119                                     rename_src[i].p->one);
120         }
121         promisor_remote_get_direct(options->repo, to_fetch.oid, to_fetch.nr);
122         oid_array_clear(&to_fetch);
123 }
124
125 static int estimate_similarity(struct repository *r,
126                                struct diff_filespec *src,
127                                struct diff_filespec *dst,
128                                int minimum_score,
129                                int skip_unmodified)
130 {
131         /* src points at a file that existed in the original tree (or
132          * optionally a file in the destination tree) and dst points
133          * at a newly created file.  They may be quite similar, in which
134          * case we want to say src is renamed to dst or src is copied into
135          * dst, and then some edit has been applied to dst.
136          *
137          * Compare them and return how similar they are, representing
138          * the score as an integer between 0 and MAX_SCORE.
139          *
140          * When there is an exact match, it is considered a better
141          * match than anything else; the destination does not even
142          * call into this function in that case.
143          */
144         unsigned long max_size, delta_size, base_size, src_copied, literal_added;
145         int score;
146         struct diff_populate_filespec_options dpf_options = {
147                 .check_size_only = 1
148         };
149         struct prefetch_options prefetch_options = {r, skip_unmodified};
150
151         if (r == the_repository && has_promisor_remote()) {
152                 dpf_options.missing_object_cb = prefetch;
153                 dpf_options.missing_object_data = &prefetch_options;
154         }
155
156         /* We deal only with regular files.  Symlink renames are handled
157          * only when they are exact matches --- in other words, no edits
158          * after renaming.
159          */
160         if (!S_ISREG(src->mode) || !S_ISREG(dst->mode))
161                 return 0;
162
163         /*
164          * Need to check that source and destination sizes are
165          * filled in before comparing them.
166          *
167          * If we already have "cnt_data" filled in, we know it's
168          * all good (avoid checking the size for zero, as that
169          * is a possible size - we really should have a flag to
170          * say whether the size is valid or not!)
171          */
172         if (!src->cnt_data &&
173             diff_populate_filespec(r, src, &dpf_options))
174                 return 0;
175         if (!dst->cnt_data &&
176             diff_populate_filespec(r, dst, &dpf_options))
177                 return 0;
178
179         max_size = ((src->size > dst->size) ? src->size : dst->size);
180         base_size = ((src->size < dst->size) ? src->size : dst->size);
181         delta_size = max_size - base_size;
182
183         /* We would not consider edits that change the file size so
184          * drastically.  delta_size must be smaller than
185          * (MAX_SCORE-minimum_score)/MAX_SCORE * min(src->size, dst->size).
186          *
187          * Note that base_size == 0 case is handled here already
188          * and the final score computation below would not have a
189          * divide-by-zero issue.
190          */
191         if (max_size * (MAX_SCORE-minimum_score) < delta_size * MAX_SCORE)
192                 return 0;
193
194         dpf_options.check_size_only = 0;
195
196         if (!src->cnt_data && diff_populate_filespec(r, src, &dpf_options))
197                 return 0;
198         if (!dst->cnt_data && diff_populate_filespec(r, dst, &dpf_options))
199                 return 0;
200
201         if (diffcore_count_changes(r, src, dst,
202                                    &src->cnt_data, &dst->cnt_data,
203                                    &src_copied, &literal_added))
204                 return 0;
205
206         /* How similar are they?
207          * what percentage of material in dst are from source?
208          */
209         if (!dst->size)
210                 score = 0; /* should not happen */
211         else
212                 score = (int)(src_copied * MAX_SCORE / max_size);
213         return score;
214 }
215
216 static void record_rename_pair(int dst_index, int src_index, int score)
217 {
218         struct diff_filepair *src = rename_src[src_index].p;
219         struct diff_filepair *dst = rename_dst[dst_index].p;
220
221         if (dst->renamed_pair)
222                 die("internal error: dst already matched.");
223
224         src->one->rename_used++;
225         src->one->count++;
226
227         rename_dst[dst_index].filespec_to_free = dst->one;
228         rename_dst[dst_index].is_rename = 1;
229
230         dst->one = src->one;
231         dst->renamed_pair = 1;
232         if (!strcmp(dst->one->path, dst->two->path))
233                 dst->score = rename_src[src_index].score;
234         else
235                 dst->score = score;
236 }
237
238 /*
239  * We sort the rename similarity matrix with the score, in descending
240  * order (the most similar first).
241  */
242 static int score_compare(const void *a_, const void *b_)
243 {
244         const struct diff_score *a = a_, *b = b_;
245
246         /* sink the unused ones to the bottom */
247         if (a->dst < 0)
248                 return (0 <= b->dst);
249         else if (b->dst < 0)
250                 return -1;
251
252         if (a->score == b->score)
253                 return b->name_score - a->name_score;
254
255         return b->score - a->score;
256 }
257
258 struct file_similarity {
259         struct hashmap_entry entry;
260         int index;
261         struct diff_filespec *filespec;
262 };
263
264 static unsigned int hash_filespec(struct repository *r,
265                                   struct diff_filespec *filespec)
266 {
267         if (!filespec->oid_valid) {
268                 if (diff_populate_filespec(r, filespec, NULL))
269                         return 0;
270                 hash_object_file(r->hash_algo, filespec->data, filespec->size,
271                                  "blob", &filespec->oid);
272         }
273         return oidhash(&filespec->oid);
274 }
275
276 static int find_identical_files(struct hashmap *srcs,
277                                 int dst_index,
278                                 struct diff_options *options)
279 {
280         int renames = 0;
281         struct diff_filespec *target = rename_dst[dst_index].p->two;
282         struct file_similarity *p, *best = NULL;
283         int i = 100, best_score = -1;
284         unsigned int hash = hash_filespec(options->repo, target);
285
286         /*
287          * Find the best source match for specified destination.
288          */
289         p = hashmap_get_entry_from_hash(srcs, hash, NULL,
290                                         struct file_similarity, entry);
291         hashmap_for_each_entry_from(srcs, p, entry) {
292                 int score;
293                 struct diff_filespec *source = p->filespec;
294
295                 /* False hash collision? */
296                 if (!oideq(&source->oid, &target->oid))
297                         continue;
298                 /* Non-regular files? If so, the modes must match! */
299                 if (!S_ISREG(source->mode) || !S_ISREG(target->mode)) {
300                         if (source->mode != target->mode)
301                                 continue;
302                 }
303                 /* Give higher scores to sources that haven't been used already */
304                 score = !source->rename_used;
305                 if (source->rename_used && options->detect_rename != DIFF_DETECT_COPY)
306                         continue;
307                 score += basename_same(source, target);
308                 if (score > best_score) {
309                         best = p;
310                         best_score = score;
311                         if (score == 2)
312                                 break;
313                 }
314
315                 /* Too many identical alternatives? Pick one */
316                 if (!--i)
317                         break;
318         }
319         if (best) {
320                 record_rename_pair(dst_index, best->index, MAX_SCORE);
321                 renames++;
322         }
323         return renames;
324 }
325
326 static void insert_file_table(struct repository *r,
327                               struct hashmap *table, int index,
328                               struct diff_filespec *filespec)
329 {
330         struct file_similarity *entry = xmalloc(sizeof(*entry));
331
332         entry->index = index;
333         entry->filespec = filespec;
334
335         hashmap_entry_init(&entry->entry, hash_filespec(r, filespec));
336         hashmap_add(table, &entry->entry);
337 }
338
339 /*
340  * Find exact renames first.
341  *
342  * The first round matches up the up-to-date entries,
343  * and then during the second round we try to match
344  * cache-dirty entries as well.
345  */
346 static int find_exact_renames(struct diff_options *options)
347 {
348         int i, renames = 0;
349         struct hashmap file_table;
350
351         /* Add all sources to the hash table in reverse order, because
352          * later on they will be retrieved in LIFO order.
353          */
354         hashmap_init(&file_table, NULL, NULL, rename_src_nr);
355         for (i = rename_src_nr-1; i >= 0; i--)
356                 insert_file_table(options->repo,
357                                   &file_table, i,
358                                   rename_src[i].p->one);
359
360         /* Walk the destinations and find best source match */
361         for (i = 0; i < rename_dst_nr; i++)
362                 renames += find_identical_files(&file_table, i, options);
363
364         /* Free the hash data structure and entries */
365         hashmap_clear_and_free(&file_table, struct file_similarity, entry);
366
367         return renames;
368 }
369
370 struct dir_rename_info {
371         struct strintmap idx_map;
372         struct strmap dir_rename_guess;
373         struct strmap *dir_rename_count;
374         struct strintmap *relevant_source_dirs;
375         unsigned setup;
376 };
377
378 static char *get_dirname(const char *filename)
379 {
380         char *slash = strrchr(filename, '/');
381         return slash ? xstrndup(filename, slash - filename) : xstrdup("");
382 }
383
384 static void dirname_munge(char *filename)
385 {
386         char *slash = strrchr(filename, '/');
387         if (!slash)
388                 slash = filename;
389         *slash = '\0';
390 }
391
392 static const char *get_highest_rename_path(struct strintmap *counts)
393 {
394         int highest_count = 0;
395         const char *highest_destination_dir = NULL;
396         struct hashmap_iter iter;
397         struct strmap_entry *entry;
398
399         strintmap_for_each_entry(counts, &iter, entry) {
400                 const char *destination_dir = entry->key;
401                 intptr_t count = (intptr_t)entry->value;
402                 if (count > highest_count) {
403                         highest_count = count;
404                         highest_destination_dir = destination_dir;
405                 }
406         }
407         return highest_destination_dir;
408 }
409
410 static void increment_count(struct dir_rename_info *info,
411                             char *old_dir,
412                             char *new_dir)
413 {
414         struct strintmap *counts;
415         struct strmap_entry *e;
416
417         /* Get the {new_dirs -> counts} mapping using old_dir */
418         e = strmap_get_entry(info->dir_rename_count, old_dir);
419         if (e) {
420                 counts = e->value;
421         } else {
422                 counts = xmalloc(sizeof(*counts));
423                 strintmap_init_with_options(counts, 0, NULL, 1);
424                 strmap_put(info->dir_rename_count, old_dir, counts);
425         }
426
427         /* Increment the count for new_dir */
428         strintmap_incr(counts, new_dir, 1);
429 }
430
431 static void update_dir_rename_counts(struct dir_rename_info *info,
432                                      struct strintmap *dirs_removed,
433                                      const char *oldname,
434                                      const char *newname)
435 {
436         char *old_dir = xstrdup(oldname);
437         char *new_dir = xstrdup(newname);
438         char new_dir_first_char = new_dir[0];
439         int first_time_in_loop = 1;
440
441         if (!info->setup)
442                 /*
443                  * info->setup is 0 here in two cases: (1) all auxiliary
444                  * vars (like dirs_removed) were NULL so
445                  * initialize_dir_rename_info() returned early, or (2)
446                  * either break detection or copy detection are active so
447                  * that we never called initialize_dir_rename_info().  In
448                  * the former case, we don't have enough info to know if
449                  * directories were renamed (because dirs_removed lets us
450                  * know about a necessary prerequisite, namely if they were
451                  * removed), and in the latter, we don't care about
452                  * directory renames or find_basename_matches.
453                  *
454                  * This matters because both basename and inexact matching
455                  * will also call update_dir_rename_counts().  In either of
456                  * the above two cases info->dir_rename_counts will not
457                  * have been properly initialized which prevents us from
458                  * updating it, but in these two cases we don't care about
459                  * dir_rename_counts anyway, so we can just exit early.
460                  */
461                 return;
462
463         while (1) {
464                 /* Get old_dir, skip if its directory isn't relevant. */
465                 dirname_munge(old_dir);
466                 if (info->relevant_source_dirs &&
467                     !strintmap_contains(info->relevant_source_dirs, old_dir))
468                         break;
469
470                 /* Get new_dir */
471                 dirname_munge(new_dir);
472
473                 /*
474                  * When renaming
475                  *   "a/b/c/d/e/foo.c" -> "a/b/some/thing/else/e/foo.c"
476                  * then this suggests that both
477                  *   a/b/c/d/e/ => a/b/some/thing/else/e/
478                  *   a/b/c/d/   => a/b/some/thing/else/
479                  * so we want to increment counters for both.  We do NOT,
480                  * however, also want to suggest that there was the following
481                  * rename:
482                  *   a/b/c/ => a/b/some/thing/
483                  * so we need to quit at that point.
484                  *
485                  * Note the when first_time_in_loop, we only strip off the
486                  * basename, and we don't care if that's different.
487                  */
488                 if (!first_time_in_loop) {
489                         char *old_sub_dir = strchr(old_dir, '\0')+1;
490                         char *new_sub_dir = strchr(new_dir, '\0')+1;
491                         if (!*new_dir) {
492                                 /*
493                                  * Special case when renaming to root directory,
494                                  * i.e. when new_dir == "".  In this case, we had
495                                  * something like
496                                  *    a/b/subdir => subdir
497                                  * and so dirname_munge() sets things up so that
498                                  *    old_dir = "a/b\0subdir\0"
499                                  *    new_dir = "\0ubdir\0"
500                                  * We didn't have a '/' to overwrite a '\0' onto
501                                  * in new_dir, so we have to compare differently.
502                                  */
503                                 if (new_dir_first_char != old_sub_dir[0] ||
504                                     strcmp(old_sub_dir+1, new_sub_dir))
505                                         break;
506                         } else {
507                                 if (strcmp(old_sub_dir, new_sub_dir))
508                                         break;
509                         }
510                 }
511
512                 if (strintmap_contains(dirs_removed, old_dir))
513                         increment_count(info, old_dir, new_dir);
514                 else
515                         break;
516
517                 /* If we hit toplevel directory ("") for old or new dir, quit */
518                 if (!*old_dir || !*new_dir)
519                         break;
520
521                 first_time_in_loop = 0;
522         }
523
524         /* Free resources we don't need anymore */
525         free(old_dir);
526         free(new_dir);
527 }
528
529 static void initialize_dir_rename_info(struct dir_rename_info *info,
530                                        struct strintmap *relevant_sources,
531                                        struct strintmap *dirs_removed,
532                                        struct strmap *dir_rename_count)
533 {
534         struct hashmap_iter iter;
535         struct strmap_entry *entry;
536         int i;
537
538         if (!dirs_removed && !relevant_sources) {
539                 info->setup = 0;
540                 return;
541         }
542         info->setup = 1;
543
544         info->dir_rename_count = dir_rename_count;
545         if (!info->dir_rename_count) {
546                 info->dir_rename_count = xmalloc(sizeof(*dir_rename_count));
547                 strmap_init(info->dir_rename_count);
548         }
549         strintmap_init_with_options(&info->idx_map, -1, NULL, 0);
550         strmap_init_with_options(&info->dir_rename_guess, NULL, 0);
551
552         /* Setup info->relevant_source_dirs */
553         info->relevant_source_dirs = NULL;
554         if (dirs_removed || !relevant_sources) {
555                 info->relevant_source_dirs = dirs_removed; /* might be NULL */
556         } else {
557                 info->relevant_source_dirs = xmalloc(sizeof(struct strintmap));
558                 strintmap_init(info->relevant_source_dirs, 0 /* unused */);
559                 strintmap_for_each_entry(relevant_sources, &iter, entry) {
560                         char *dirname = get_dirname(entry->key);
561                         if (!dirs_removed ||
562                             strintmap_contains(dirs_removed, dirname))
563                                 strintmap_set(info->relevant_source_dirs,
564                                               dirname, 0 /* value irrelevant */);
565                         free(dirname);
566                 }
567         }
568
569         /*
570          * Loop setting up both info->idx_map, and doing setup of
571          * info->dir_rename_count.
572          */
573         for (i = 0; i < rename_dst_nr; ++i) {
574                 /*
575                  * For non-renamed files, make idx_map contain mapping of
576                  *   filename -> index (index within rename_dst, that is)
577                  */
578                 if (!rename_dst[i].is_rename) {
579                         char *filename = rename_dst[i].p->two->path;
580                         strintmap_set(&info->idx_map, filename, i);
581                         continue;
582                 }
583
584                 /*
585                  * For everything else (i.e. renamed files), make
586                  * dir_rename_count contain a map of a map:
587                  *   old_directory -> {new_directory -> count}
588                  * In other words, for every pair look at the directories for
589                  * the old filename and the new filename and count how many
590                  * times that pairing occurs.
591                  */
592                 update_dir_rename_counts(info, dirs_removed,
593                                          rename_dst[i].p->one->path,
594                                          rename_dst[i].p->two->path);
595         }
596
597         /*
598          * Now we collapse
599          *    dir_rename_count: old_directory -> {new_directory -> count}
600          * down to
601          *    dir_rename_guess: old_directory -> best_new_directory
602          * where best_new_directory is the one with the highest count.
603          */
604         strmap_for_each_entry(info->dir_rename_count, &iter, entry) {
605                 /* entry->key is source_dir */
606                 struct strintmap *counts = entry->value;
607                 char *best_newdir;
608
609                 best_newdir = xstrdup(get_highest_rename_path(counts));
610                 strmap_put(&info->dir_rename_guess, entry->key,
611                            best_newdir);
612         }
613 }
614
615 void partial_clear_dir_rename_count(struct strmap *dir_rename_count)
616 {
617         struct hashmap_iter iter;
618         struct strmap_entry *entry;
619
620         strmap_for_each_entry(dir_rename_count, &iter, entry) {
621                 struct strintmap *counts = entry->value;
622                 strintmap_clear(counts);
623         }
624         strmap_partial_clear(dir_rename_count, 1);
625 }
626
627 static void cleanup_dir_rename_info(struct dir_rename_info *info,
628                                     struct strintmap *dirs_removed,
629                                     int keep_dir_rename_count)
630 {
631         struct hashmap_iter iter;
632         struct strmap_entry *entry;
633         struct string_list to_remove = STRING_LIST_INIT_NODUP;
634         int i;
635
636         if (!info->setup)
637                 return;
638
639         /* idx_map */
640         strintmap_clear(&info->idx_map);
641
642         /* dir_rename_guess */
643         strmap_clear(&info->dir_rename_guess, 1);
644
645         /* relevant_source_dirs */
646         if (info->relevant_source_dirs &&
647             info->relevant_source_dirs != dirs_removed) {
648                 strintmap_clear(info->relevant_source_dirs);
649                 FREE_AND_NULL(info->relevant_source_dirs);
650         }
651
652         /* dir_rename_count */
653         if (!keep_dir_rename_count) {
654                 partial_clear_dir_rename_count(info->dir_rename_count);
655                 strmap_clear(info->dir_rename_count, 1);
656                 FREE_AND_NULL(info->dir_rename_count);
657                 return;
658         }
659
660         /*
661          * Although dir_rename_count was passed in
662          * diffcore_rename_extended() and we want to keep it around and
663          * return it to that caller, we first want to remove any data
664          * associated with directories that weren't renamed.
665          */
666         strmap_for_each_entry(info->dir_rename_count, &iter, entry) {
667                 const char *source_dir = entry->key;
668                 struct strintmap *counts = entry->value;
669
670                 if (!strintmap_get(dirs_removed, source_dir)) {
671                         string_list_append(&to_remove, source_dir);
672                         strintmap_clear(counts);
673                         continue;
674                 }
675         }
676         for (i = 0; i < to_remove.nr; ++i)
677                 strmap_remove(info->dir_rename_count,
678                               to_remove.items[i].string, 1);
679         string_list_clear(&to_remove, 0);
680 }
681
682 static const char *get_basename(const char *filename)
683 {
684         /*
685          * gitbasename() has to worry about special drives, multiple
686          * directory separator characters, trailing slashes, NULL or
687          * empty strings, etc.  We only work on filenames as stored in
688          * git, and thus get to ignore all those complications.
689          */
690         const char *base = strrchr(filename, '/');
691         return base ? base + 1 : filename;
692 }
693
694 static int idx_possible_rename(char *filename, struct dir_rename_info *info)
695 {
696         /*
697          * Our comparison of files with the same basename (see
698          * find_basename_matches() below), is only helpful when after exact
699          * rename detection we have exactly one file with a given basename
700          * among the rename sources and also only exactly one file with
701          * that basename among the rename destinations.  When we have
702          * multiple files with the same basename in either set, we do not
703          * know which to compare against.  However, there are some
704          * filenames that occur in large numbers (particularly
705          * build-related filenames such as 'Makefile', '.gitignore', or
706          * 'build.gradle' that potentially exist within every single
707          * subdirectory), and for performance we want to be able to quickly
708          * find renames for these files too.
709          *
710          * The reason basename comparisons are a useful heuristic was that it
711          * is common for people to move files across directories while keeping
712          * their filename the same.  If we had a way of determining or even
713          * making a good educated guess about which directory these non-unique
714          * basename files had moved the file to, we could check it.
715          * Luckily...
716          *
717          * When an entire directory is in fact renamed, we have two factors
718          * helping us out:
719          *   (a) the original directory disappeared giving us a hint
720          *       about when we can apply an extra heuristic.
721          *   (a) we often have several files within that directory and
722          *       subdirectories that are renamed without changes
723          * So, rules for a heuristic:
724          *   (0) If there basename matches are non-unique (the condition under
725          *       which this function is called) AND
726          *   (1) the directory in which the file was found has disappeared
727          *       (i.e. dirs_removed is non-NULL and has a relevant entry) THEN
728          *   (2) use exact renames of files within the directory to determine
729          *       where the directory is likely to have been renamed to.  IF
730          *       there is at least one exact rename from within that
731          *       directory, we can proceed.
732          *   (3) If there are multiple places the directory could have been
733          *       renamed to based on exact renames, ignore all but one of them.
734          *       Just use the destination with the most renames going to it.
735          *   (4) Check if applying that directory rename to the original file
736          *       would result in a destination filename that is in the
737          *       potential rename set.  If so, return the index of the
738          *       destination file (the index within rename_dst).
739          *   (5) Compare the original file and returned destination for
740          *       similarity, and if they are sufficiently similar, record the
741          *       rename.
742          *
743          * This function, idx_possible_rename(), is only responsible for (4).
744          * The conditions/steps in (1)-(3) are handled via setting up
745          * dir_rename_count and dir_rename_guess in
746          * initialize_dir_rename_info().  Steps (0) and (5) are handled by
747          * the caller of this function.
748          */
749         char *old_dir, *new_dir;
750         struct strbuf new_path = STRBUF_INIT;
751         int idx;
752
753         if (!info->setup)
754                 return -1;
755
756         old_dir = get_dirname(filename);
757         new_dir = strmap_get(&info->dir_rename_guess, old_dir);
758         free(old_dir);
759         if (!new_dir)
760                 return -1;
761
762         strbuf_addstr(&new_path, new_dir);
763         strbuf_addch(&new_path, '/');
764         strbuf_addstr(&new_path, get_basename(filename));
765
766         idx = strintmap_get(&info->idx_map, new_path.buf);
767         strbuf_release(&new_path);
768         return idx;
769 }
770
771 static int find_basename_matches(struct diff_options *options,
772                                  int minimum_score,
773                                  struct dir_rename_info *info,
774                                  struct strintmap *relevant_sources,
775                                  struct strintmap *dirs_removed)
776 {
777         /*
778          * When I checked in early 2020, over 76% of file renames in linux
779          * just moved files to a different directory but kept the same
780          * basename.  gcc did that with over 64% of renames, gecko did it
781          * with over 79%, and WebKit did it with over 89%.
782          *
783          * Therefore we can bypass the normal exhaustive NxM matrix
784          * comparison of similarities between all potential rename sources
785          * and destinations by instead using file basename as a hint (i.e.
786          * the portion of the filename after the last '/'), checking for
787          * similarity between files with the same basename, and if we find
788          * a pair that are sufficiently similar, record the rename pair and
789          * exclude those two from the NxM matrix.
790          *
791          * This *might* cause us to find a less than optimal pairing (if
792          * there is another file that we are even more similar to but has a
793          * different basename).  Given the huge performance advantage
794          * basename matching provides, and given the frequency with which
795          * people use the same basename in real world projects, that's a
796          * trade-off we are willing to accept when doing just rename
797          * detection.
798          *
799          * If someone wants copy detection that implies they are willing to
800          * spend more cycles to find similarities between files, so it may
801          * be less likely that this heuristic is wanted.  If someone is
802          * doing break detection, that means they do not want filename
803          * similarity to imply any form of content similiarity, and thus
804          * this heuristic would definitely be incompatible.
805          */
806
807         int i, renames = 0;
808         struct strintmap sources;
809         struct strintmap dests;
810
811         /*
812          * The prefeteching stuff wants to know if it can skip prefetching
813          * blobs that are unmodified...and will then do a little extra work
814          * to verify that the oids are indeed different before prefetching.
815          * Unmodified blobs are only relevant when doing copy detection;
816          * when limiting to rename detection, diffcore_rename[_extended]()
817          * will never be called with unmodified source paths fed to us, so
818          * the extra work necessary to check if rename_src entries are
819          * unmodified would be a small waste.
820          */
821         int skip_unmodified = 0;
822
823         /*
824          * Create maps of basename -> fullname(s) for remaining sources and
825          * dests.
826          */
827         strintmap_init_with_options(&sources, -1, NULL, 0);
828         strintmap_init_with_options(&dests, -1, NULL, 0);
829         for (i = 0; i < rename_src_nr; ++i) {
830                 char *filename = rename_src[i].p->one->path;
831                 const char *base;
832
833                 /* exact renames removed in remove_unneeded_paths_from_src() */
834                 assert(!rename_src[i].p->one->rename_used);
835
836                 /* Record index within rename_src (i) if basename is unique */
837                 base = get_basename(filename);
838                 if (strintmap_contains(&sources, base))
839                         strintmap_set(&sources, base, -1);
840                 else
841                         strintmap_set(&sources, base, i);
842         }
843         for (i = 0; i < rename_dst_nr; ++i) {
844                 char *filename = rename_dst[i].p->two->path;
845                 const char *base;
846
847                 if (rename_dst[i].is_rename)
848                         continue; /* involved in exact match already. */
849
850                 /* Record index within rename_dst (i) if basename is unique */
851                 base = get_basename(filename);
852                 if (strintmap_contains(&dests, base))
853                         strintmap_set(&dests, base, -1);
854                 else
855                         strintmap_set(&dests, base, i);
856         }
857
858         /* Now look for basename matchups and do similarity estimation */
859         for (i = 0; i < rename_src_nr; ++i) {
860                 char *filename = rename_src[i].p->one->path;
861                 const char *base = NULL;
862                 intptr_t src_index;
863                 intptr_t dst_index;
864
865                 /* Skip irrelevant sources */
866                 if (relevant_sources &&
867                     !strintmap_contains(relevant_sources, filename))
868                         continue;
869
870                 /*
871                  * If the basename is unique among remaining sources, then
872                  * src_index will equal 'i' and we can attempt to match it
873                  * to a unique basename in the destinations.  Otherwise,
874                  * use directory rename heuristics, if possible.
875                  */
876                 base = get_basename(filename);
877                 src_index = strintmap_get(&sources, base);
878                 assert(src_index == -1 || src_index == i);
879
880                 if (strintmap_contains(&dests, base)) {
881                         struct diff_filespec *one, *two;
882                         int score;
883
884                         /* Find a matching destination, if possible */
885                         dst_index = strintmap_get(&dests, base);
886                         if (src_index == -1 || dst_index == -1) {
887                                 src_index = i;
888                                 dst_index = idx_possible_rename(filename, info);
889                         }
890                         if (dst_index == -1)
891                                 continue;
892
893                         /* Ignore this dest if already used in a rename */
894                         if (rename_dst[dst_index].is_rename)
895                                 continue; /* already used previously */
896
897                         /* Estimate the similarity */
898                         one = rename_src[src_index].p->one;
899                         two = rename_dst[dst_index].p->two;
900                         score = estimate_similarity(options->repo, one, two,
901                                                     minimum_score, skip_unmodified);
902
903                         /* If sufficiently similar, record as rename pair */
904                         if (score < minimum_score)
905                                 continue;
906                         record_rename_pair(dst_index, src_index, score);
907                         renames++;
908                         update_dir_rename_counts(info, dirs_removed,
909                                                  one->path, two->path);
910
911                         /*
912                          * Found a rename so don't need text anymore; if we
913                          * didn't find a rename, the filespec_blob would get
914                          * re-used when doing the matrix of comparisons.
915                          */
916                         diff_free_filespec_blob(one);
917                         diff_free_filespec_blob(two);
918                 }
919         }
920
921         strintmap_clear(&sources);
922         strintmap_clear(&dests);
923
924         return renames;
925 }
926
927 #define NUM_CANDIDATE_PER_DST 4
928 static void record_if_better(struct diff_score m[], struct diff_score *o)
929 {
930         int i, worst;
931
932         /* find the worst one */
933         worst = 0;
934         for (i = 1; i < NUM_CANDIDATE_PER_DST; i++)
935                 if (score_compare(&m[i], &m[worst]) > 0)
936                         worst = i;
937
938         /* is it better than the worst one? */
939         if (score_compare(&m[worst], o) > 0)
940                 m[worst] = *o;
941 }
942
943 /*
944  * Returns:
945  * 0 if we are under the limit;
946  * 1 if we need to disable inexact rename detection;
947  * 2 if we would be under the limit if we were given -C instead of -C -C.
948  */
949 static int too_many_rename_candidates(int num_destinations, int num_sources,
950                                       struct diff_options *options)
951 {
952         int rename_limit = options->rename_limit;
953         int i, limited_sources;
954
955         options->needed_rename_limit = 0;
956
957         /*
958          * This basically does a test for the rename matrix not
959          * growing larger than a "rename_limit" square matrix, ie:
960          *
961          *    num_destinations * num_sources > rename_limit * rename_limit
962          *
963          * We use st_mult() to check overflow conditions; in the
964          * exceptional circumstance that size_t isn't large enough to hold
965          * the multiplication, the system won't be able to allocate enough
966          * memory for the matrix anyway.
967          */
968         if (rename_limit <= 0)
969                 rename_limit = 32767;
970         if (st_mult(num_destinations, num_sources)
971             <= st_mult(rename_limit, rename_limit))
972                 return 0;
973
974         options->needed_rename_limit =
975                 num_sources > num_destinations ? num_sources : num_destinations;
976
977         /* Are we running under -C -C? */
978         if (!options->flags.find_copies_harder)
979                 return 1;
980
981         /* Would we bust the limit if we were running under -C? */
982         for (limited_sources = i = 0; i < num_sources; i++) {
983                 if (diff_unmodified_pair(rename_src[i].p))
984                         continue;
985                 limited_sources++;
986         }
987         if (st_mult(num_destinations, limited_sources)
988             <= st_mult(rename_limit, rename_limit))
989                 return 2;
990         return 1;
991 }
992
993 static int find_renames(struct diff_score *mx,
994                         int dst_cnt,
995                         int minimum_score,
996                         int copies,
997                         struct dir_rename_info *info,
998                         struct strintmap *dirs_removed)
999 {
1000         int count = 0, i;
1001
1002         for (i = 0; i < dst_cnt * NUM_CANDIDATE_PER_DST; i++) {
1003                 struct diff_rename_dst *dst;
1004
1005                 if ((mx[i].dst < 0) ||
1006                     (mx[i].score < minimum_score))
1007                         break; /* there is no more usable pair. */
1008                 dst = &rename_dst[mx[i].dst];
1009                 if (dst->is_rename)
1010                         continue; /* already done, either exact or fuzzy. */
1011                 if (!copies && rename_src[mx[i].src].p->one->rename_used)
1012                         continue;
1013                 record_rename_pair(mx[i].dst, mx[i].src, mx[i].score);
1014                 count++;
1015                 update_dir_rename_counts(info, dirs_removed,
1016                                          rename_src[mx[i].src].p->one->path,
1017                                          rename_dst[mx[i].dst].p->two->path);
1018         }
1019         return count;
1020 }
1021
1022 static void remove_unneeded_paths_from_src(int detecting_copies,
1023                                            struct strintmap *interesting)
1024 {
1025         int i, new_num_src;
1026
1027         if (detecting_copies && !interesting)
1028                 return; /* nothing to remove */
1029         if (break_idx)
1030                 return; /* culling incompatible with break detection */
1031
1032         /*
1033          * Note on reasons why we cull unneeded sources but not destinations:
1034          *   1) Pairings are stored in rename_dst (not rename_src), which we
1035          *      need to keep around.  So, we just can't cull rename_dst even
1036          *      if we wanted to.  But doing so wouldn't help because...
1037          *
1038          *   2) There is a matrix pairwise comparison that follows the
1039          *      "Performing inexact rename detection" progress message.
1040          *      Iterating over the destinations is done in the outer loop,
1041          *      hence we only iterate over each of those once and we can
1042          *      easily skip the outer loop early if the destination isn't
1043          *      relevant.  That's only one check per destination path to
1044          *      skip.
1045          *
1046          *      By contrast, the sources are iterated in the inner loop; if
1047          *      we check whether a source can be skipped, then we'll be
1048          *      checking it N separate times, once for each destination.
1049          *      We don't want to have to iterate over known-not-needed
1050          *      sources N times each, so avoid that by removing the sources
1051          *      from rename_src here.
1052          */
1053         for (i = 0, new_num_src = 0; i < rename_src_nr; i++) {
1054                 struct diff_filespec *one = rename_src[i].p->one;
1055
1056                 /*
1057                  * renames are stored in rename_dst, so if a rename has
1058                  * already been detected using this source, we can just
1059                  * remove the source knowing rename_dst has its info.
1060                  */
1061                 if (!detecting_copies && one->rename_used)
1062                         continue;
1063
1064                 /* If we don't care about the source path, skip it */
1065                 if (interesting && !strintmap_contains(interesting, one->path))
1066                         continue;
1067
1068                 if (new_num_src < i)
1069                         memcpy(&rename_src[new_num_src], &rename_src[i],
1070                                sizeof(struct diff_rename_src));
1071                 new_num_src++;
1072         }
1073
1074         rename_src_nr = new_num_src;
1075 }
1076
1077 static void handle_early_known_dir_renames(struct dir_rename_info *info,
1078                                            struct strintmap *relevant_sources,
1079                                            struct strintmap *dirs_removed)
1080 {
1081         /*
1082          * Not yet implemented; directory renames are determined via an
1083          * aggregate of all renames under them and using a "majority wins"
1084          * rule.  The fact that "majority wins", though, means we don't need
1085          * all the renames under the given directory, we only need enough to
1086          * ensure we have a majority.
1087          *
1088          * For now, we don't have enough information to know if we have a
1089          * majority after exact renames and basename-guided rename detection,
1090          * so just return early without doing any extra filtering.
1091          */
1092         return;
1093 }
1094
1095 void diffcore_rename_extended(struct diff_options *options,
1096                               struct strintmap *relevant_sources,
1097                               struct strintmap *dirs_removed,
1098                               struct strmap *dir_rename_count)
1099 {
1100         int detect_rename = options->detect_rename;
1101         int minimum_score = options->rename_score;
1102         struct diff_queue_struct *q = &diff_queued_diff;
1103         struct diff_queue_struct outq;
1104         struct diff_score *mx;
1105         int i, j, rename_count, skip_unmodified = 0;
1106         int num_destinations, dst_cnt;
1107         int num_sources, want_copies;
1108         struct progress *progress = NULL;
1109         struct dir_rename_info info;
1110
1111         trace2_region_enter("diff", "setup", options->repo);
1112         info.setup = 0;
1113         assert(!dir_rename_count || strmap_empty(dir_rename_count));
1114         want_copies = (detect_rename == DIFF_DETECT_COPY);
1115         if (dirs_removed && (break_idx || want_copies))
1116                 BUG("dirs_removed incompatible with break/copy detection");
1117         if (break_idx && relevant_sources)
1118                 BUG("break detection incompatible with source specification");
1119         if (!minimum_score)
1120                 minimum_score = DEFAULT_RENAME_SCORE;
1121
1122         for (i = 0; i < q->nr; i++) {
1123                 struct diff_filepair *p = q->queue[i];
1124                 if (!DIFF_FILE_VALID(p->one)) {
1125                         if (!DIFF_FILE_VALID(p->two))
1126                                 continue; /* unmerged */
1127                         else if (options->single_follow &&
1128                                  strcmp(options->single_follow, p->two->path))
1129                                 continue; /* not interested */
1130                         else if (!options->flags.rename_empty &&
1131                                  is_empty_blob_oid(&p->two->oid))
1132                                 continue;
1133                         else if (add_rename_dst(p) < 0) {
1134                                 warning("skipping rename detection, detected"
1135                                         " duplicate destination '%s'",
1136                                         p->two->path);
1137                                 goto cleanup;
1138                         }
1139                 }
1140                 else if (!options->flags.rename_empty &&
1141                          is_empty_blob_oid(&p->one->oid))
1142                         continue;
1143                 else if (!DIFF_PAIR_UNMERGED(p) && !DIFF_FILE_VALID(p->two)) {
1144                         /*
1145                          * If the source is a broken "delete", and
1146                          * they did not really want to get broken,
1147                          * that means the source actually stays.
1148                          * So we increment the "rename_used" score
1149                          * by one, to indicate ourselves as a user
1150                          */
1151                         if (p->broken_pair && !p->score)
1152                                 p->one->rename_used++;
1153                         register_rename_src(p);
1154                 }
1155                 else if (want_copies) {
1156                         /*
1157                          * Increment the "rename_used" score by
1158                          * one, to indicate ourselves as a user.
1159                          */
1160                         p->one->rename_used++;
1161                         register_rename_src(p);
1162                 }
1163         }
1164         trace2_region_leave("diff", "setup", options->repo);
1165         if (rename_dst_nr == 0 || rename_src_nr == 0)
1166                 goto cleanup; /* nothing to do */
1167
1168         trace2_region_enter("diff", "exact renames", options->repo);
1169         /*
1170          * We really want to cull the candidates list early
1171          * with cheap tests in order to avoid doing deltas.
1172          */
1173         rename_count = find_exact_renames(options);
1174         trace2_region_leave("diff", "exact renames", options->repo);
1175
1176         /* Did we only want exact renames? */
1177         if (minimum_score == MAX_SCORE)
1178                 goto cleanup;
1179
1180         num_sources = rename_src_nr;
1181
1182         if (want_copies || break_idx) {
1183                 /*
1184                  * Cull sources:
1185                  *   - remove ones corresponding to exact renames
1186                  *   - remove ones not found in relevant_sources
1187                  */
1188                 trace2_region_enter("diff", "cull after exact", options->repo);
1189                 remove_unneeded_paths_from_src(want_copies, relevant_sources);
1190                 trace2_region_leave("diff", "cull after exact", options->repo);
1191         } else {
1192                 /* Determine minimum score to match basenames */
1193                 double factor = 0.5;
1194                 char *basename_factor = getenv("GIT_BASENAME_FACTOR");
1195                 int min_basename_score;
1196
1197                 if (basename_factor)
1198                         factor = strtol(basename_factor, NULL, 10)/100.0;
1199                 assert(factor >= 0.0 && factor <= 1.0);
1200                 min_basename_score = minimum_score +
1201                         (int)(factor * (MAX_SCORE - minimum_score));
1202
1203                 /*
1204                  * Cull sources:
1205                  *   - remove ones involved in renames (found via exact match)
1206                  */
1207                 trace2_region_enter("diff", "cull after exact", options->repo);
1208                 remove_unneeded_paths_from_src(want_copies, NULL);
1209                 trace2_region_leave("diff", "cull after exact", options->repo);
1210
1211                 /* Preparation for basename-driven matching. */
1212                 trace2_region_enter("diff", "dir rename setup", options->repo);
1213                 initialize_dir_rename_info(&info, relevant_sources,
1214                                            dirs_removed, dir_rename_count);
1215                 trace2_region_leave("diff", "dir rename setup", options->repo);
1216
1217                 /* Utilize file basenames to quickly find renames. */
1218                 trace2_region_enter("diff", "basename matches", options->repo);
1219                 rename_count += find_basename_matches(options,
1220                                                       min_basename_score,
1221                                                       &info,
1222                                                       relevant_sources,
1223                                                       dirs_removed);
1224                 trace2_region_leave("diff", "basename matches", options->repo);
1225
1226                 /*
1227                  * Cull sources, again:
1228                  *   - remove ones involved in renames (found via basenames)
1229                  *   - remove ones not found in relevant_sources
1230                  * and
1231                  *   - remove ones in relevant_sources which are needed only
1232                  *     for directory renames IF no ancestory directory
1233                  *     actually needs to know any more individual path
1234                  *     renames under them
1235                  */
1236                 trace2_region_enter("diff", "cull basename", options->repo);
1237                 remove_unneeded_paths_from_src(want_copies, relevant_sources);
1238                 handle_early_known_dir_renames(&info, relevant_sources,
1239                                                dirs_removed);
1240                 trace2_region_leave("diff", "cull basename", options->repo);
1241         }
1242
1243         /* Calculate how many rename destinations are left */
1244         num_destinations = (rename_dst_nr - rename_count);
1245         num_sources = rename_src_nr; /* rename_src_nr reflects lower number */
1246
1247         /* All done? */
1248         if (!num_destinations || !num_sources)
1249                 goto cleanup;
1250
1251         switch (too_many_rename_candidates(num_destinations, num_sources,
1252                                            options)) {
1253         case 1:
1254                 goto cleanup;
1255         case 2:
1256                 options->degraded_cc_to_c = 1;
1257                 skip_unmodified = 1;
1258                 break;
1259         default:
1260                 break;
1261         }
1262
1263         trace2_region_enter("diff", "inexact renames", options->repo);
1264         if (options->show_rename_progress) {
1265                 progress = start_delayed_progress(
1266                                 _("Performing inexact rename detection"),
1267                                 (uint64_t)num_destinations * (uint64_t)num_sources);
1268         }
1269
1270         mx = xcalloc(st_mult(NUM_CANDIDATE_PER_DST, num_destinations),
1271                      sizeof(*mx));
1272         for (dst_cnt = i = 0; i < rename_dst_nr; i++) {
1273                 struct diff_filespec *two = rename_dst[i].p->two;
1274                 struct diff_score *m;
1275
1276                 if (rename_dst[i].is_rename)
1277                         continue; /* exact or basename match already handled */
1278
1279                 m = &mx[dst_cnt * NUM_CANDIDATE_PER_DST];
1280                 for (j = 0; j < NUM_CANDIDATE_PER_DST; j++)
1281                         m[j].dst = -1;
1282
1283                 for (j = 0; j < rename_src_nr; j++) {
1284                         struct diff_filespec *one = rename_src[j].p->one;
1285                         struct diff_score this_src;
1286
1287                         assert(!one->rename_used || want_copies || break_idx);
1288
1289                         if (skip_unmodified &&
1290                             diff_unmodified_pair(rename_src[j].p))
1291                                 continue;
1292
1293                         this_src.score = estimate_similarity(options->repo,
1294                                                              one, two,
1295                                                              minimum_score,
1296                                                              skip_unmodified);
1297                         this_src.name_score = basename_same(one, two);
1298                         this_src.dst = i;
1299                         this_src.src = j;
1300                         record_if_better(m, &this_src);
1301                         /*
1302                          * Once we run estimate_similarity,
1303                          * We do not need the text anymore.
1304                          */
1305                         diff_free_filespec_blob(one);
1306                         diff_free_filespec_blob(two);
1307                 }
1308                 dst_cnt++;
1309                 display_progress(progress,
1310                                  (uint64_t)dst_cnt * (uint64_t)num_sources);
1311         }
1312         stop_progress(&progress);
1313
1314         /* cost matrix sorted by most to least similar pair */
1315         STABLE_QSORT(mx, dst_cnt * NUM_CANDIDATE_PER_DST, score_compare);
1316
1317         rename_count += find_renames(mx, dst_cnt, minimum_score, 0,
1318                                      &info, dirs_removed);
1319         if (want_copies)
1320                 rename_count += find_renames(mx, dst_cnt, minimum_score, 1,
1321                                              &info, dirs_removed);
1322         free(mx);
1323         trace2_region_leave("diff", "inexact renames", options->repo);
1324
1325  cleanup:
1326         /* At this point, we have found some renames and copies and they
1327          * are recorded in rename_dst.  The original list is still in *q.
1328          */
1329         trace2_region_enter("diff", "write back to queue", options->repo);
1330         DIFF_QUEUE_CLEAR(&outq);
1331         for (i = 0; i < q->nr; i++) {
1332                 struct diff_filepair *p = q->queue[i];
1333                 struct diff_filepair *pair_to_free = NULL;
1334
1335                 if (DIFF_PAIR_UNMERGED(p)) {
1336                         diff_q(&outq, p);
1337                 }
1338                 else if (!DIFF_FILE_VALID(p->one) && DIFF_FILE_VALID(p->two)) {
1339                         /* Creation */
1340                         diff_q(&outq, p);
1341                 }
1342                 else if (DIFF_FILE_VALID(p->one) && !DIFF_FILE_VALID(p->two)) {
1343                         /*
1344                          * Deletion
1345                          *
1346                          * We would output this delete record if:
1347                          *
1348                          * (1) this is a broken delete and the counterpart
1349                          *     broken create remains in the output; or
1350                          * (2) this is not a broken delete, and rename_dst
1351                          *     does not have a rename/copy to move p->one->path
1352                          *     out of existence.
1353                          *
1354                          * Otherwise, the counterpart broken create
1355                          * has been turned into a rename-edit; or
1356                          * delete did not have a matching create to
1357                          * begin with.
1358                          */
1359                         if (DIFF_PAIR_BROKEN(p)) {
1360                                 /* broken delete */
1361                                 struct diff_rename_dst *dst = locate_rename_dst(p);
1362                                 if (!dst)
1363                                         BUG("tracking failed somehow; failed to find associated dst for broken pair");
1364                                 if (dst->is_rename)
1365                                         /* counterpart is now rename/copy */
1366                                         pair_to_free = p;
1367                         }
1368                         else {
1369                                 if (p->one->rename_used)
1370                                         /* this path remains */
1371                                         pair_to_free = p;
1372                         }
1373
1374                         if (!pair_to_free)
1375                                 diff_q(&outq, p);
1376                 }
1377                 else if (!diff_unmodified_pair(p))
1378                         /* all the usual ones need to be kept */
1379                         diff_q(&outq, p);
1380                 else
1381                         /* no need to keep unmodified pairs; FIXME: remove earlier? */
1382                         pair_to_free = p;
1383
1384                 if (pair_to_free)
1385                         diff_free_filepair(pair_to_free);
1386         }
1387         diff_debug_queue("done copying original", &outq);
1388
1389         free(q->queue);
1390         *q = outq;
1391         diff_debug_queue("done collapsing", q);
1392
1393         for (i = 0; i < rename_dst_nr; i++)
1394                 if (rename_dst[i].filespec_to_free)
1395                         free_filespec(rename_dst[i].filespec_to_free);
1396
1397         cleanup_dir_rename_info(&info, dirs_removed, dir_rename_count != NULL);
1398         FREE_AND_NULL(rename_dst);
1399         rename_dst_nr = rename_dst_alloc = 0;
1400         FREE_AND_NULL(rename_src);
1401         rename_src_nr = rename_src_alloc = 0;
1402         if (break_idx) {
1403                 strintmap_clear(break_idx);
1404                 FREE_AND_NULL(break_idx);
1405         }
1406         trace2_region_leave("diff", "write back to queue", options->repo);
1407         return;
1408 }
1409
1410 void diffcore_rename(struct diff_options *options)
1411 {
1412         diffcore_rename_extended(options, NULL, NULL, NULL);
1413 }