[PATCH 3/6] submodule: remove add_oid_to_argv
- From
- Brandon Williams <bmwill@google.com>
- Date
- Apr 28, 2017, 23:53 UTC
- Message-ID
- <20170428235402.162251-4-bmwill@google.com>
- In-Reply-To
- <20170428235402.162251-1-bmwill@google.com>
The function 'add_oid_to_argv()' provides the same functionality as 'append_oid_to_argv()'. Remove this duplicate function and instead use 'append_oid_to_argv()' where 'add_oid_to_argv()' was previously used.
Signed-off-by: Brandon Williams <bmwill@google.com> --- submodule.c | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/submodule.c b/submodule.c index 46abd52b1..7baa28ae0 100644 --- a/submodule.c +++ b/submodule.c @@ -970,12 +970,6 @@ void check_for_new_submodule_commits(struct object_id *oid) oid_array_append(&ref_tips_after_fetch, oid); } -static int add_oid_to_argv(const struct object_id *oid, void *data) -{ - argv_array_push(data, oid_to_hex(oid)); - return 0; -} - static void calculate_changed_submodule_paths(void) { struct rev_info rev; @@ -989,10 +983,10 @@ static void calculate_changed_submodule_paths(void) init_revisions(&rev, NULL); argv_array_push(&argv, "--"); /* argv[0] program name */ oid_array_for_each_unique(&ref_tips_after_fetch, - add_oid_to_argv, &argv); + append_oid_to_argv, &argv); argv_array_push(&argv, "--not"); oid_array_for_each_unique(&ref_tips_before_fetch, - add_oid_to_argv, &argv); + append_oid_to_argv, &argv); setup_revisions(argv.argc, argv.argv, &rev, NULL); if (prepare_revision_walk(&rev)) die("revision walk setup failed");
-- 2.13.0.rc0.306.g87b477812d-goog