@matheusbsbs / matheus-bagliano / commits / fe88f9f91f
Fork 0

blame: drop unused parameter from maybe_changed_path

We don't use the "parent" parameter at all (probably because the bloom filter for a commit is always defined against a single parent anyway). Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Jeff King committed Apr 23, 2020 at 17:03 UTC fe88f9f91ffcb7a4dbb83b17e0b62c12409a6047
1 file changed +1 -3
blame.c
+1 -3
@@ -1263,7 +1263,6 @@ struct blame_bloom_data {
1263 static int bloom_count_queries = 0;
1264 static int bloom_count_no = 0;
1265 static int maybe_changed_path(struct repository *r,
1266 - struct commit *parent,
1266 struct blame_origin *origin,
1267 struct blame_bloom_data *bd)
1268 {
@@ -1355,8 +1354,7 @@ static struct blame_origin *find_origin(struct repository *r,
1354 if (origin->commit->parents &&
1355 !oidcmp(&parent->object.oid,
1356 &origin->commit->parents->item->object.oid))
1358 - compute_diff = maybe_changed_path(r, parent,
1359 - origin, bd);
1357 + compute_diff = maybe_changed_path(r, origin, bd);
1358
1359 if (compute_diff)
1360 diff_tree_oid(get_commit_tree_oid(parent),