diff options
| author | Edward Thomson <ethomson@edwardthomson.com> | 2020-05-29 13:13:19 +0100 |
|---|---|---|
| committer | Edward Thomson <ethomson@edwardthomson.com> | 2020-06-01 22:50:28 +0100 |
| commit | 51eff5a58b95f91cbdd8e5caa750964c9f08e895 (patch) | |
| tree | 64cebe7a2646eacd6c1fdd068a17213fef39c23b /tests | |
| parent | a9746b306d32cc2d2bc5d446f6f7ae7c7068ba79 (diff) | |
| download | libgit2-51eff5a58b95f91cbdd8e5caa750964c9f08e895.tar.gz | |
strarray: we should `dispose` instead of `free`
We _dispose_ the contents of objects; we _free_ objects (and their
contents). Update `git_strarray_free` to be `git_strarray_dispose`.
`git_strarray_free` remains as a deprecated proxy function.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/network/fetchlocal.c | 28 | ||||
| -rw-r--r-- | tests/network/remote/remotes.c | 14 | ||||
| -rw-r--r-- | tests/network/remote/rename.c | 22 | ||||
| -rw-r--r-- | tests/object/tag/list.c | 4 | ||||
| -rw-r--r-- | tests/online/fetchhead.c | 2 | ||||
| -rw-r--r-- | tests/online/remotes.c | 4 | ||||
| -rw-r--r-- | tests/refs/list.c | 4 | ||||
| -rw-r--r-- | tests/refs/listall.c | 4 | ||||
| -rw-r--r-- | tests/refs/namespaces.c | 2 | ||||
| -rw-r--r-- | tests/remote/create.c | 20 | ||||
| -rw-r--r-- | tests/remote/list.c | 6 | ||||
| -rw-r--r-- | tests/submodule/add.c | 2 | ||||
| -rw-r--r-- | tests/worktree/bare.c | 4 | ||||
| -rw-r--r-- | tests/worktree/refs.c | 4 | ||||
| -rw-r--r-- | tests/worktree/worktree.c | 8 |
15 files changed, 64 insertions, 64 deletions
diff --git a/tests/network/fetchlocal.c b/tests/network/fetchlocal.c index 804308070..13f03100b 100644 --- a/tests/network/fetchlocal.c +++ b/tests/network/fetchlocal.c @@ -49,7 +49,7 @@ void test_network_fetchlocal__complete(void) cl_assert_equal_i(19, (int)refnames.count); cl_assert(callcount > 0); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(origin); git_repository_free(repo); } @@ -77,7 +77,7 @@ void test_network_fetchlocal__prune(void) cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(19, (int)refnames.count); cl_assert(callcount > 0); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(origin); cl_git_pass(git_reference_lookup(&ref, remote_repo, "refs/heads/br2")); @@ -90,7 +90,7 @@ void test_network_fetchlocal__prune(void) cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(18, (int)refnames.count); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(origin); cl_git_pass(git_reference_lookup(&ref, remote_repo, "refs/heads/packed")); @@ -103,7 +103,7 @@ void test_network_fetchlocal__prune(void) cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(17, (int)refnames.count); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(origin); git_repository_free(repo); @@ -168,7 +168,7 @@ void test_network_fetchlocal__prune_overlapping(void) assert_ref_exists(repo, "refs/remotes/origin/pr/42"); cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(20, (int)refnames.count); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); cl_git_pass(git_config_delete_multivar(config, "remote.origin.fetch", "refs")); cl_git_pass(git_config_set_multivar(config, "remote.origin.fetch", "^$", "refs/pull/*/head:refs/remotes/origin/pr/*")); @@ -183,7 +183,7 @@ void test_network_fetchlocal__prune_overlapping(void) assert_ref_exists(repo, "refs/remotes/origin/pr/42"); cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(20, (int)refnames.count); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); cl_git_pass(git_config_delete_multivar(config, "remote.origin.fetch", "refs")); cl_git_pass(git_config_set_multivar(config, "remote.origin.fetch", "^$", "refs/heads/*:refs/remotes/origin/*")); @@ -195,7 +195,7 @@ void test_network_fetchlocal__prune_overlapping(void) cl_git_pass(git_remote_fetch(origin, NULL, &options, NULL)); git_config_free(config); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(origin); git_repository_free(repo); } @@ -224,7 +224,7 @@ void test_network_fetchlocal__fetchprune(void) cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(19, (int)refnames.count); cl_assert(callcount > 0); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(origin); cl_git_pass(git_reference_lookup(&ref, remote_repo, "refs/heads/br2")); @@ -237,7 +237,7 @@ void test_network_fetchlocal__fetchprune(void) cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(18, (int)refnames.count); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(origin); cl_git_pass(git_reference_lookup(&ref, remote_repo, "refs/heads/packed")); @@ -253,7 +253,7 @@ void test_network_fetchlocal__fetchprune(void) cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(17, (int)refnames.count); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(origin); git_repository_free(repo); @@ -333,13 +333,13 @@ void test_network_fetchlocal__partial(void) cl_git_pass(git_remote_create(&origin, repo, GIT_REMOTE_ORIGIN, url)); cl_git_pass(git_remote_fetch(origin, NULL, &options, NULL)); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(20, (int)refnames.count); /* 18 remote + 1 local */ cl_assert(callcount > 0); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(origin); } @@ -420,7 +420,7 @@ void test_network_fetchlocal__multi_remotes(void) cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(33, (int)refnames.count); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); cl_git_pass(git_remote_set_url(repo, "test_with_pushurl", cl_git_fixture_url("testrepo.git"))); cl_git_pass(git_remote_lookup(&test2, repo, "test_with_pushurl")); @@ -429,7 +429,7 @@ void test_network_fetchlocal__multi_remotes(void) cl_git_pass(git_reference_list(&refnames, repo)); cl_assert_equal_i(45, (int)refnames.count); - git_strarray_free(&refnames); + git_strarray_dispose(&refnames); git_remote_free(test); git_remote_free(test2); } diff --git a/tests/network/remote/remotes.c b/tests/network/remote/remotes.c index a29281d37..0694a6f66 100644 --- a/tests/network/remote/remotes.c +++ b/tests/network/remote/remotes.c @@ -61,7 +61,7 @@ static int urlresolve_callback(git_buf *url_resolved, const char *url, int direc cl_assert(strcmp(url, "git://github.com/libgit2/libgit2") == 0); cl_assert(strcmp(payload, "payload") == 0); cl_assert(url_resolved->size == 0); - + if (direction == GIT_DIRECTION_PUSH) git_buf_sets(url_resolved, "pushresolve"); if (direction == GIT_DIRECTION_FETCH) @@ -215,11 +215,11 @@ void test_network_remote_remotes__dup(void) cl_git_pass(git_remote_get_fetch_refspecs(&array, _remote)); cl_assert_equal_i(1, (int)array.count); cl_assert_equal_s("+refs/heads/*:refs/remotes/test/*", array.strings[0]); - git_strarray_free(&array); + git_strarray_dispose(&array); cl_git_pass(git_remote_get_push_refspecs(&array, _remote)); cl_assert_equal_i(0, (int)array.count); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(dup); } @@ -318,7 +318,7 @@ void test_network_remote_remotes__list(void) cl_git_pass(git_remote_list(&list, _repo)); cl_assert(list.count == 5); - git_strarray_free(&list); + git_strarray_dispose(&list); cl_git_pass(git_repository_config(&cfg, _repo)); @@ -330,7 +330,7 @@ void test_network_remote_remotes__list(void) cl_git_pass(git_remote_list(&list, _repo)); cl_assert(list.count == 7); - git_strarray_free(&list); + git_strarray_dispose(&list); git_config_free(cfg); } @@ -466,13 +466,13 @@ void test_network_remote_remotes__query_refspecs(void) for (i = 0; i < 3; i++) { cl_assert_equal_s(fetch_refspecs[i], array.strings[i]); } - git_strarray_free(&array); + git_strarray_dispose(&array); cl_git_pass(git_remote_get_push_refspecs(&array, remote)); for (i = 0; i < 3; i++) { cl_assert_equal_s(push_refspecs[i], array.strings[i]); } - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); git_remote_delete(_repo, "test"); diff --git a/tests/network/remote/rename.c b/tests/network/remote/rename.c index 0449575d7..1fd2affba 100644 --- a/tests/network/remote/rename.c +++ b/tests/network/remote/rename.c @@ -25,7 +25,7 @@ void test_network_remote_rename__renaming_a_remote_moves_related_configuration_s cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "just/renamed")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); assert_config_entry_existence(_repo, "remote.test.fetch", false); assert_config_entry_existence(_repo, "remote.just/renamed.fetch", true); @@ -39,7 +39,7 @@ void test_network_remote_rename__renaming_a_remote_updates_branch_related_config cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "just/renamed")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); assert_config_entry_value(_repo, "branch.master.remote", "just/renamed"); } @@ -50,7 +50,7 @@ void test_network_remote_rename__renaming_a_remote_updates_default_fetchrefspec( cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "just/renamed")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); assert_config_entry_value(_repo, "remote.just/renamed.fetch", "+refs/heads/*:refs/remotes/just/renamed/*"); } @@ -71,7 +71,7 @@ void test_network_remote_rename__renaming_a_remote_without_a_fetchrefspec_doesnt cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "just/renamed")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); assert_config_entry_existence(_repo, "remote.just/renamed.fetch", false); } @@ -90,11 +90,11 @@ void test_network_remote_rename__renaming_a_remote_notifies_of_non_default_fetch cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "just/renamed")); cl_assert_equal_i(1, problems.count); cl_assert_equal_s("+refs/*:refs/*", problems.strings[0]); - git_strarray_free(&problems); + git_strarray_dispose(&problems); assert_config_entry_value(_repo, "remote.just/renamed.fetch", "+refs/*:refs/*"); - git_strarray_free(&problems); + git_strarray_dispose(&problems); } void test_network_remote_rename__new_name_can_contain_dots(void) @@ -103,7 +103,7 @@ void test_network_remote_rename__new_name_can_contain_dots(void) cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "just.renamed")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); assert_config_entry_existence(_repo, "remote.just.renamed.fetch", true); } @@ -126,7 +126,7 @@ void test_network_remote_rename__renamed_name_is_persisted(void) cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "just/renamed")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); cl_git_pass(git_repository_open(&another_repo, "testrepo.git")); cl_git_pass(git_remote_lookup(&renamed, _repo, "just/renamed")); @@ -154,7 +154,7 @@ void test_network_remote_rename__renaming_a_remote_moves_the_underlying_referenc cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "just/renamed")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); cl_assert_equal_i(GIT_ENOTFOUND, git_reference_lookup(&underlying, _repo, "refs/remotes/test/master")); cl_git_pass(git_reference_lookup(&underlying, _repo, "refs/remotes/just/renamed/master")); @@ -176,7 +176,7 @@ void test_network_remote_rename__overwrite_ref_in_target(void) cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "renamed")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); /* make sure there's only one remote-tracking branch */ cl_git_pass(git_branch_iterator_new(&iter, _repo, GIT_BRANCH_REMOTE)); @@ -214,7 +214,7 @@ void test_network_remote_rename__symref_head(void) cl_git_pass(git_remote_rename(&problems, _repo, _remote_name, "renamed")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); cl_git_pass(git_vector_init(&refs, 2, (git_vector_cmp) git_reference_cmp)); cl_git_pass(git_branch_iterator_new(&iter, _repo, GIT_BRANCH_REMOTE)); diff --git a/tests/object/tag/list.c b/tests/object/tag/list.c index b8d507fe7..8a1a2d26d 100644 --- a/tests/object/tag/list.c +++ b/tests/object/tag/list.c @@ -50,7 +50,7 @@ static void ensure_tag_pattern_match(git_repository *repo, cl_assert_equal_i((int)sucessfully_found, (int)data->expected_matches); exit: - git_strarray_free(&tag_list); + git_strarray_dispose(&tag_list); cl_git_pass(error); } @@ -74,7 +74,7 @@ void test_object_tag_list__list_all(void) cl_assert_equal_i((int)tag_list.count, 6); - git_strarray_free(&tag_list); + git_strarray_dispose(&tag_list); } static const struct pattern_match_t matches[] = { diff --git a/tests/online/fetchhead.c b/tests/online/fetchhead.c index 32e7419ae..b61993138 100644 --- a/tests/online/fetchhead.c +++ b/tests/online/fetchhead.c @@ -43,7 +43,7 @@ static size_t count_references(void) cl_git_pass(git_reference_list(&array, g_repo)); refs = array.count; - git_strarray_free(&array); + git_strarray_dispose(&array); return refs; } diff --git a/tests/online/remotes.c b/tests/online/remotes.c index d79eb1f59..f7fe4142f 100644 --- a/tests/online/remotes.c +++ b/tests/online/remotes.c @@ -32,7 +32,7 @@ void test_online_remotes__single_branch(void) } cl_assert_equal_i(1, count); - git_strarray_free(&refs); + git_strarray_dispose(&refs); cl_git_pass(git_remote_lookup(&remote, repo, "origin")); cl_git_pass(git_remote_get_fetch_refspecs(&refs, remote)); @@ -40,7 +40,7 @@ void test_online_remotes__single_branch(void) cl_assert_equal_i(1, refs.count); cl_assert_equal_s(REFSPEC, refs.strings[0]); - git_strarray_free(&refs); + git_strarray_dispose(&refs); git_remote_free(remote); git_repository_free(repo); } diff --git a/tests/refs/list.c b/tests/refs/list.c index 725d38161..8085ff84b 100644 --- a/tests/refs/list.c +++ b/tests/refs/list.c @@ -38,7 +38,7 @@ void test_refs_list__all(void) * loose, but we only list it once */ cl_assert_equal_i((int)ref_list.count, 19); - git_strarray_free(&ref_list); + git_strarray_dispose(&ref_list); } void test_refs_list__do_not_retrieve_references_which_name_end_with_a_lock_extension(void) @@ -53,5 +53,5 @@ void test_refs_list__do_not_retrieve_references_which_name_end_with_a_lock_exten cl_git_pass(git_reference_list(&ref_list, g_repo)); cl_assert_equal_i((int)ref_list.count, 19); - git_strarray_free(&ref_list); + git_strarray_dispose(&ref_list); } diff --git a/tests/refs/listall.c b/tests/refs/listall.c index c696fbb2e..9da8d1ac3 100644 --- a/tests/refs/listall.c +++ b/tests/refs/listall.c @@ -16,7 +16,7 @@ static void ensure_no_refname_starts_with_a_forward_slash(const char *path) for (i = 0; i < ref_list.count; i++) cl_assert(git__prefixcmp(ref_list.strings[i], "/") != 0); - git_strarray_free(&ref_list); + git_strarray_dispose(&ref_list); git_repository_free(repo); } @@ -42,6 +42,6 @@ void test_refs_listall__from_repository_with_no_trailing_newline(void) cl_assert(ref_list.count > 0); - git_strarray_free(&ref_list); + git_strarray_dispose(&ref_list); git_repository_free(repo); } diff --git a/tests/refs/namespaces.c b/tests/refs/namespaces.c index bb6bb1ce0..19456b5a4 100644 --- a/tests/refs/namespaces.c +++ b/tests/refs/namespaces.c @@ -32,5 +32,5 @@ void test_refs_namespaces__namespace_doesnt_show_normal_refs(void) cl_git_pass(git_repository_set_namespace(g_repo, "namespace")); cl_git_pass(git_reference_list(&ref_list, g_repo)); cl_assert_equal_i(0, ref_list.count); - git_strarray_free(&ref_list); + git_strarray_dispose(&ref_list); } diff --git a/tests/remote/create.c b/tests/remote/create.c index 510962314..f92be9dfc 100644 --- a/tests/remote/create.c +++ b/tests/remote/create.c @@ -117,7 +117,7 @@ void test_remote_create__with_fetchspec(void) cl_assert_equal_i(1, array.count); cl_assert_equal_i(section_count + 2, count_config_entries_match(_repo, "remote\\.")); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); } @@ -132,7 +132,7 @@ void test_remote_create__with_empty_fetchspec(void) cl_assert_equal_i(0, array.count); cl_assert_equal_i(section_count + 1, count_config_entries_match(_repo, "remote\\.")); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); } @@ -167,7 +167,7 @@ void test_remote_create__anonymous(void) cl_assert_equal_i(0, array.count); cl_assert_equal_i(section_count, count_config_entries_match(_repo, "remote\\.")); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); } @@ -195,7 +195,7 @@ void test_remote_create__detached(void) cl_assert_equal_i(0, array.count); cl_assert_equal_i(section_count, count_config_entries_match(_repo, "remote\\.")); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); } @@ -225,7 +225,7 @@ void test_remote_create__with_opts_named(void) cl_assert_equal_i(1, array.count); cl_assert_equal_s("+refs/heads/*:refs/remotes/test-new/*", array.strings[0]); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); } @@ -248,7 +248,7 @@ void test_remote_create__with_opts_named_and_fetchspec(void) cl_assert_equal_i(1, array.count); cl_assert_equal_s("+refs/*:refs/*", array.strings[0]); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); } @@ -270,7 +270,7 @@ void test_remote_create__with_opts_named_no_fetchspec(void) cl_git_pass(git_remote_get_fetch_refspecs(&array, remote)); cl_assert_equal_i(0, array.count); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); } @@ -290,7 +290,7 @@ void test_remote_create__with_opts_anonymous(void) cl_git_pass(git_remote_get_fetch_refspecs(&array, remote)); cl_assert_equal_i(0, array.count); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); } @@ -308,7 +308,7 @@ void test_remote_create__with_opts_detached(void) cl_git_pass(git_remote_get_fetch_refspecs(&array, remote)); cl_assert_equal_i(0, array.count); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); @@ -320,7 +320,7 @@ void test_remote_create__with_opts_detached(void) cl_git_pass(git_remote_get_fetch_refspecs(&array, remote)); cl_assert_equal_i(0, array.count); - git_strarray_free(&array); + git_strarray_dispose(&array); git_remote_free(remote); } diff --git a/tests/remote/list.c b/tests/remote/list.c index 5abb95106..4a6be3d1b 100644 --- a/tests/remote/list.c +++ b/tests/remote/list.c @@ -25,17 +25,17 @@ void test_remote_list__always_checks_disk_config(void) cl_git_pass(git_remote_list(&remotes, _repo)); cl_assert_equal_sz(remotes.count, 1); - git_strarray_free(&remotes); + git_strarray_dispose(&remotes); cl_git_pass(git_remote_create(&remote, _repo, "valid-name", TEST_URL)); cl_git_pass(git_remote_list(&remotes, _repo)); cl_assert_equal_sz(remotes.count, 2); - git_strarray_free(&remotes); + git_strarray_dispose(&remotes); cl_git_pass(git_remote_list(&remotes, repo)); cl_assert_equal_sz(remotes.count, 2); - git_strarray_free(&remotes); + git_strarray_dispose(&remotes); git_repository_free(repo); git_remote_free(remote); diff --git a/tests/submodule/add.c b/tests/submodule/add.c index f4d1e3b79..fc458f826 100644 --- a/tests/submodule/add.c +++ b/tests/submodule/add.c @@ -90,7 +90,7 @@ void test_submodule_add__url_relative(void) /* make sure we don't default to origin - rename origin -> test_remote */ cl_git_pass(git_remote_rename(&problems, g_repo, "origin", "test_remote")); cl_assert_equal_i(0, problems.count); - git_strarray_free(&problems); + git_strarray_dispose(&problems); cl_git_fail(git_remote_lookup(&remote, g_repo, "origin")); cl_git_pass( diff --git a/tests/worktree/bare.c b/tests/worktree/bare.c index 2a0e88239..7234dfffd 100644 --- a/tests/worktree/bare.c +++ b/tests/worktree/bare.c @@ -28,7 +28,7 @@ void test_worktree_bare__list(void) cl_git_pass(git_worktree_list(&wts, g_repo)); cl_assert_equal_i(wts.count, 0); - git_strarray_free(&wts); + git_strarray_dispose(&wts); } void test_worktree_bare__add(void) @@ -48,7 +48,7 @@ void test_worktree_bare__add(void) cl_assert_equal_i(0, git_repository_is_bare(wtrepo)); cl_assert_equal_i(1, git_repository_is_worktree(wtrepo)); - git_strarray_free(&wts); + git_strarray_dispose(&wts); git_worktree_free(wt); git_repository_free(wtrepo); } diff --git a/tests/worktree/refs.c b/tests/worktree/refs.c index e62149d2e..501255274 100644 --- a/tests/worktree/refs.c +++ b/tests/worktree/refs.c @@ -56,8 +56,8 @@ void test_worktree_refs__list(void) } exit: - git_strarray_free(&refs); - git_strarray_free(&wtrefs); + git_strarray_dispose(&refs); + git_strarray_dispose(&wtrefs); cl_git_pass(error); } diff --git a/tests/worktree/worktree.c b/tests/worktree/worktree.c index 5e99dbf61..716d0aa0a 100644 --- a/tests/worktree/worktree.c +++ b/tests/worktree/worktree.c @@ -30,7 +30,7 @@ void test_worktree_worktree__list(void) cl_assert_equal_i(wts.count, 1); cl_assert_equal_s(wts.strings[0], "testrepo-worktree"); - git_strarray_free(&wts); + git_strarray_dispose(&wts); } void test_worktree_worktree__list_with_invalid_worktree_dirs(void) @@ -61,7 +61,7 @@ void test_worktree_worktree__list_with_invalid_worktree_dirs(void) cl_git_pass(git_worktree_list(&wts, fixture.worktree)); cl_assert_equal_i(wts.count, 1); cl_assert_equal_s(wts.strings[0], "testrepo-worktree"); - git_strarray_free(&wts); + git_strarray_dispose(&wts); for (j = 0; j < ARRAY_SIZE(filesets[i]); j++) { git_buf_truncate(&path, len); @@ -81,7 +81,7 @@ void test_worktree_worktree__list_in_worktree_repo(void) cl_assert_equal_i(wts.count, 1); cl_assert_equal_s(wts.strings[0], "testrepo-worktree"); - git_strarray_free(&wts); + git_strarray_dispose(&wts); } void test_worktree_worktree__list_without_worktrees(void) @@ -380,7 +380,7 @@ void test_worktree_worktree__name(void) cl_git_pass(git_worktree_lookup(&wt, fixture.repo, "testrepo-worktree")); cl_assert_equal_s(git_worktree_name(wt), "testrepo-worktree"); - + git_worktree_free(wt); } |
