Re: [PATCH v3 3/4] diffcore-rename: treat a rename_limit of 0 as unlimited
From: Junio C Hamano <hidden>
Date: 2021-07-15 23:17:18
"Elijah Newren via GitGitGadget" [off-list ref] writes:
quoted hunk
diff --git a/diffcore-rename.c b/diffcore-rename.c index 3375e24659e..513ba7b05f1 100644 --- a/diffcore-rename.c +++ b/diffcore-rename.c@@ -1021,7 +1021,7 @@ static int too_many_rename_candidates(int num_destinations, int num_sources, * memory for the matrix anyway. */ if (rename_limit <= 0) - rename_limit = 32767; + return 0; /* treat as unlimited */
OK. As this short-cuts, the impact a non-positive value may have on all the remainder of the function (like limit squared must be larger than the matrix) would not have to be compensated for. Very simple and clean.
if (st_mult(num_destinations, num_sources) <= st_mult(rename_limit, rename_limit)) return 0;