Commit b595b385dfbed63146f61185f1074d1334432bdd

Vicent Martí 2013-09-03T04:11:07

Merge pull request #1814 from libgit2/is-empty-fix Fix incorrect precedence within git_repository_is_empty()

diff --git a/src/repository.c b/src/repository.c
index 99ac56e..eae22ce 100644
--- a/src/repository.c
+++ b/src/repository.c
@@ -1495,24 +1495,20 @@ static int repo_contains_no_reference(git_repository *repo)
 int git_repository_is_empty(git_repository *repo)
 {
 	git_reference *head = NULL;
-	int error;
+	int is_empty = 0;
 
 	if (git_reference_lookup(&head, repo, GIT_HEAD_FILE) < 0)
 		return -1;
 
-	if (!((error = git_reference_type(head)) == GIT_REF_SYMBOLIC))
-		goto cleanup;
-
-	if (!(error = (strcmp(
-		git_reference_symbolic_target(head),
-		GIT_REFS_HEADS_DIR "master") == 0)))
-			goto cleanup;
+	if (git_reference_type(head) == GIT_REF_SYMBOLIC)
+		is_empty =
+			(strcmp(git_reference_symbolic_target(head),
+					GIT_REFS_HEADS_DIR "master") == 0) &&
+			repo_contains_no_reference(repo);
 
-	error = repo_contains_no_reference(repo);
-
-cleanup:
 	git_reference_free(head);
-	return error < 0 ? -1 : error;
+
+	return is_empty;
 }
 
 const char *git_repository_path(git_repository *repo)