Skip to content

Commit

Permalink
copytrace: remove the outdated comments
Browse files Browse the repository at this point in the history
Summary:
We removed those path similarity related constants in D50411161, but didn't remove the
corresponding comments. This diff removes them.

Reviewed By: sggutier

Differential Revision: D67487767

fbshipit-source-id: d0a6be025de5965e64e2316ff1daf279cbb0ab17
  • Loading branch information
zzl0 authored and facebook-github-bot committed Dec 20, 2024
1 parent c1628cc commit 4493daf
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions eden/scm/lib/copytrace/src/utils.rs
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,6 @@ const DEFAULT_SIMILARITY_THRESHOLD: f32 = 0.8;
/// Maximum rename edit cost determines whether we treat two files as a rename
const DEFAULT_MAX_EDIT_COST: u64 = 1000;

/// Contants for path similarity score. The actually value does not matter
/// here, we are more care about the ordering. Based on Git community's data,
/// ~70%+ of renames have the same basename:
/// https://github.com/git/git/blob/74cc1aa55f30ed76424a0e7226ab519aa6265061/diffcore-rename.c#L904-L907
/// Computes the similarity score between file paths.
///
/// The score will be in range [0.0, 1.0]. Higher number means more similar.
Expand Down

0 comments on commit 4493daf

Please sign in to comment.