diff options
author | Edward Thomson <ethomson@edwardthomson.com> | 2018-12-19 10:48:16 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-12-19 10:48:16 -0400 |
commit | 77f1460fcf6de77aebd55fc5ef30a663df38a55e (patch) | |
tree | 5e0df8bc28df9d8ac48087c07391fe6c4f3ba094 | |
parent | bc2196575467e7fef30a445d5ae840350ed0467e (diff) | |
parent | 8dde7e1137a05bdb6f6309f68ae6b0cfa61e2a71 (diff) | |
download | libgit2-77f1460fcf6de77aebd55fc5ef30a663df38a55e.tar.gz |
Merge pull request #4915 from pks-t/pks/refdb-minor-refactorings
refdb_fs: refactor error handling in `refdb_reflog_fs__delete`
-rw-r--r-- | src/refdb_fs.c | 23 |
1 files changed, 11 insertions, 12 deletions
diff --git a/src/refdb_fs.c b/src/refdb_fs.c index 46f552e91..1eebf5174 100644 --- a/src/refdb_fs.c +++ b/src/refdb_fs.c @@ -2046,28 +2046,27 @@ cleanup: static int refdb_reflog_fs__delete(git_refdb_backend *_backend, const char *name) { - int error; + refdb_fs_backend *backend = (refdb_fs_backend *) _backend; git_buf path = GIT_BUF_INIT; - - git_repository *repo; - refdb_fs_backend *backend; + int error; assert(_backend && name); - backend = (refdb_fs_backend *) _backend; - repo = backend->repo; + if ((error = retrieve_reflog_path(&path, backend->repo, name)) < 0) + goto out; - error = retrieve_reflog_path(&path, repo, name); + if (!git_path_exists(path.ptr)) + goto out; - if (!error && git_path_exists(path.ptr)) { - error = p_unlink(path.ptr); - refdb_fs_backend__try_delete_empty_ref_hierarchie(backend, name, true); - } + if ((error = p_unlink(path.ptr)) < 0) + goto out; + refdb_fs_backend__try_delete_empty_ref_hierarchie(backend, name, true); + +out: git_buf_dispose(&path); return error; - } int git_refdb_backend_fs( |