diff options
author | Edward Thomson <ethomson@edwardthomson.com> | 2022-01-21 19:43:42 -0500 |
---|---|---|
committer | Edward Thomson <ethomson@edwardthomson.com> | 2022-01-22 19:09:35 -0500 |
commit | 05f061522f9eda57c4082dea0c50eae1dda7763d (patch) | |
tree | fc7ec8e46b8d3e00a77c5af31ef9f2c7e95b9fe3 | |
parent | 3f0b00f7d4c8ec61c9133dfff12771b04fc558f3 (diff) | |
download | libgit2-05f061522f9eda57c4082dea0c50eae1dda7763d.tar.gz |
oid: avoid `tostr_s` in many places
The `git_oid_tostr_s` helper is indeed helpful, unless you are using
printf debugging (by inserting more `git_oid_tostr_s` calls) shortly
after using it. Avoid it before invoking complex functions.
-rw-r--r-- | src/branch.c | 5 | ||||
-rw-r--r-- | src/reset.c | 5 |
2 files changed, 8 insertions, 2 deletions
diff --git a/src/branch.c b/src/branch.c index 03892a4b0..2e29af99d 100644 --- a/src/branch.c +++ b/src/branch.c @@ -123,7 +123,10 @@ int git_branch_create( const git_commit *commit, int force) { - return create_branch(ref_out, repository, branch_name, commit, git_oid_tostr_s(git_commit_id(commit)), force); + char commit_id[GIT_OID_HEXSZ + 1]; + + git_oid_tostr(commit_id, GIT_OID_HEXSZ + 1, git_commit_id(commit)); + return create_branch(ref_out, repository, branch_name, commit, commit_id, force); } int git_branch_create_from_annotated( diff --git a/src/reset.c b/src/reset.c index b8327fe5e..e0d942e5e 100644 --- a/src/reset.c +++ b/src/reset.c @@ -188,7 +188,10 @@ int git_reset( git_reset_t reset_type, const git_checkout_options *checkout_opts) { - return reset(repo, target, git_oid_tostr_s(git_object_id(target)), reset_type, checkout_opts); + char to[GIT_OID_HEXSZ + 1]; + + git_oid_tostr(to, GIT_OID_HEXSZ + 1, git_object_id(target)); + return reset(repo, target, to, reset_type, checkout_opts); } int git_reset_from_annotated( |