diff options
author | Edward Thomson <ethomson@microsoft.com> | 2014-03-12 09:43:53 -0700 |
---|---|---|
committer | Edward Thomson <ethomson@microsoft.com> | 2014-03-20 09:25:09 -0700 |
commit | d9fdee6e4cb87e4531d9ddba92b44e5323e794da (patch) | |
tree | d243e3feb5fadcf806e674fd786ff8330ae7d95b /src | |
parent | 5aa2ac6de1622308884fd542f7b29e6810a0e98e (diff) | |
download | libgit2-d9fdee6e4cb87e4531d9ddba92b44e5323e794da.tar.gz |
Remove `git_merge_result` as it's now unnecessary
Diffstat (limited to 'src')
-rw-r--r-- | src/merge.c | 48 | ||||
-rw-r--r-- | src/merge.h | 10 |
2 files changed, 1 insertions, 57 deletions
diff --git a/src/merge.c b/src/merge.c index 589cf43de..66b8be684 100644 --- a/src/merge.c +++ b/src/merge.c @@ -2548,14 +2548,12 @@ done: } int git_merge( - git_merge_result **out, git_repository *repo, const git_merge_head **their_heads, size_t their_heads_len, const git_merge_options *merge_opts, const git_checkout_options *given_checkout_opts) { - git_merge_result *result; git_reference *our_ref = NULL; git_checkout_options checkout_opts; git_merge_head *ancestor_head = NULL, *our_head = NULL; @@ -2564,18 +2562,13 @@ int git_merge( size_t i; int error = 0; - assert(out && repo && their_heads); - - *out = NULL; + assert(repo && their_heads); if (their_heads_len != 1) { giterr_set(GITERR_MERGE, "Can only merge a single branch"); return -1; } - result = git__calloc(1, sizeof(git_merge_result)); - GITERR_CHECK_ALLOC(result); - their_trees = git__calloc(their_heads_len, sizeof(git_tree *)); GITERR_CHECK_ALLOC(their_trees); @@ -2610,16 +2603,12 @@ int git_merge( (error = git_checkout_index(repo, index_repo, &checkout_opts)) < 0) goto on_error; - result->index = index_new; - - *out = result; goto done; on_error: merge_state_cleanup(repo); git_index_free(index_new); - git__free(result); done: git_index_free(index_repo); @@ -2640,41 +2629,6 @@ done: return error; } -/* Merge result data */ - -int git_merge_result_is_uptodate(git_merge_result *merge_result) -{ - assert(merge_result); - - return merge_result->is_uptodate; -} - -int git_merge_result_is_fastforward(git_merge_result *merge_result) -{ - assert(merge_result); - - return merge_result->is_fastforward; -} - -int git_merge_result_fastforward_id(git_oid *out, git_merge_result *merge_result) -{ - assert(out && merge_result); - - git_oid_cpy(out, &merge_result->fastforward_oid); - return 0; -} - -void git_merge_result_free(git_merge_result *merge_result) -{ - if (merge_result == NULL) - return; - - git_index_free(merge_result->index); - merge_result->index = NULL; - - git__free(merge_result); -} - /* Merge heads are the input to merge */ static int merge_head_init( diff --git a/src/merge.h b/src/merge.h index 44c808dfa..2362da04d 100644 --- a/src/merge.h +++ b/src/merge.h @@ -120,16 +120,6 @@ struct git_merge_head { git_commit *commit; }; -/** Internal structure for merge results */ -struct git_merge_result { - bool is_uptodate; - - bool is_fastforward; - git_oid fastforward_oid; - - git_index *index; -}; - int git_merge__bases_many( git_commit_list **out, git_revwalk *walk, |