]> git.ipfire.org Git - thirdparty/git.git/commitdiff
merge-ort: drop unused "opt" parameter from merge_check_renames_reusable()
authorJeff King <peff@peff.net>
Thu, 14 Sep 2023 09:40:04 +0000 (05:40 -0400)
committerJunio C Hamano <gitster@pobox.com>
Thu, 14 Sep 2023 19:01:29 +0000 (12:01 -0700)
The merge_options parameter has never been used since the function was
introduced in 64aceb6d73 (merge-ort: add code to check for whether
cached renames can be reused, 2021-05-20). In theory some merge options
might impact our decisions here, but that has never been the case so
far.

Let's drop it to appease -Wunused-parameter; it would be easy to add
back later if we need to (there is only one caller).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
merge-ort.c

index 20eefd9b5ed20b37a5d4d8e2ec59ad4ba22bc6e7..3953c9f7453e3c4017dbcd08763ef14c46c44ba0 100644 (file)
@@ -4880,8 +4880,7 @@ static void merge_start(struct merge_options *opt, struct merge_result *result)
        trace2_region_leave("merge", "allocate/init", opt->repo);
 }
 
-static void merge_check_renames_reusable(struct merge_options *opt,
-                                        struct merge_result *result,
+static void merge_check_renames_reusable(struct merge_result *result,
                                         struct tree *merge_base,
                                         struct tree *side1,
                                         struct tree *side2)
@@ -5083,7 +5082,7 @@ void merge_incore_nonrecursive(struct merge_options *opt,
 
        trace2_region_enter("merge", "merge_start", opt->repo);
        assert(opt->ancestor != NULL);
-       merge_check_renames_reusable(opt, result, merge_base, side1, side2);
+       merge_check_renames_reusable(result, merge_base, side1, side2);
        merge_start(opt, result);
        /*
         * Record the trees used in this merge, so if there's a next merge in