]> git.ipfire.org Git - thirdparty/git.git/commitdiff
merge-ours: drop USE_THE_REPOSITORY_VARIABLE
authorSam Bostock <sam@sambostock.ca>
Fri, 6 Feb 2026 19:16:22 +0000 (19:16 +0000)
committerJunio C Hamano <gitster@pobox.com>
Fri, 6 Feb 2026 19:45:31 +0000 (11:45 -0800)
The merge-ours built-in uses the `the_repository` global to access
the repository. The project is moving away from this global in favor
of the `repo` parameter that is passed to each built-in command.
Since merge-ours is registered with RUN_SETUP, `repo` is guaranteed
to be non-NULL and can be used directly.

Drop the USE_THE_REPOSITORY_VARIABLE macro and use `repo` throughout.

While at it, remove a stray double blank line between the #include
block and the usage string.

Signed-off-by: Sam Bostock <sam@sambostock.ca>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/merge-ours.c

index 97b8a792c77eb7c2b0447d532cc19342a974ac47..2312e58ab370686b13663fe25d9894adfff57f0c 100644 (file)
@@ -8,20 +8,17 @@
  * Pretend we resolved the heads, but declare our tree trumps everybody else.
  */
 
-#define USE_THE_REPOSITORY_VARIABLE
-
 #include "git-compat-util.h"
 #include "builtin.h"
 #include "diff.h"
 
-
 static const char builtin_merge_ours_usage[] =
        "git merge-ours <base>... -- HEAD <remote>...";
 
 int cmd_merge_ours(int argc,
                   const char **argv,
                   const char *prefix UNUSED,
-                  struct repository *repo UNUSED)
+                  struct repository *repo)
 {
        show_usage_if_asked(argc, argv, builtin_merge_ours_usage);
 
@@ -30,9 +27,9 @@ int cmd_merge_ours(int argc,
         * commit.  The index must match HEAD, or this merge cannot go
         * through.
         */
-       if (repo_read_index(the_repository) < 0)
+       if (repo_read_index(repo) < 0)
                die_errno("read_cache failed");
-       if (index_differs_from(the_repository, "HEAD", NULL, 0))
+       if (index_differs_from(repo, "HEAD", NULL, 0))
                return 2;
        return 0;
 }