From 4493daf78f0257ac13d5146b263cc5f6a8606cc5 Mon Sep 17 00:00:00 2001 From: Zhaolong Zhu Date: Thu, 19 Dec 2024 18:41:56 -0800 Subject: [PATCH] copytrace: remove the outdated comments 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 --- eden/scm/lib/copytrace/src/utils.rs | 5 ----- 1 file changed, 5 deletions(-) diff --git a/eden/scm/lib/copytrace/src/utils.rs b/eden/scm/lib/copytrace/src/utils.rs index 04198896a4998..0d3d9af49de72 100644 --- a/eden/scm/lib/copytrace/src/utils.rs +++ b/eden/scm/lib/copytrace/src/utils.rs @@ -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.