diff options
author | Arkady Shapkin <arkadiy_s@inbox.ru> | 2014-09-03 01:01:25 +0400 |
---|---|---|
committer | Arkady Shapkin <arkadiy_s@inbox.ru> | 2014-09-03 01:01:25 +0400 |
commit | 5cd81bb3d88218d42dd431f2c5011dfa84c287c7 (patch) | |
tree | 43bfda14197265192576842893ecc49b17a0383d | |
parent | d4cb23e4e50688a9079ab0e3ec12cf2cc1256f04 (diff) | |
download | libgit2-5cd81bb3d88218d42dd431f2c5011dfa84c287c7.tar.gz |
Several CppCat warnings fixed
-rw-r--r-- | src/blame.c | 1 | ||||
-rw-r--r-- | src/pack.c | 3 | ||||
-rw-r--r-- | src/stash.c | 3 | ||||
-rw-r--r-- | src/transports/smart_protocol.c | 2 |
4 files changed, 3 insertions, 6 deletions
diff --git a/src/blame.c b/src/blame.c index eb977c287..2cc5e552b 100644 --- a/src/blame.c +++ b/src/blame.c @@ -316,7 +316,6 @@ static int blame_internal(git_blame *blame) ent->suspect = o; blame->ent = ent; - blame->path = blame->path; git_blame__like_git(blame, blame->options.flags); diff --git a/src/pack.c b/src/pack.c index b05aa91f1..7c1cfe03e 100644 --- a/src/pack.c +++ b/src/pack.c @@ -648,9 +648,6 @@ int git_packfile_unpack( base_type = elem->type; } - if (error < 0) - goto cleanup; - switch (base_type) { case GIT_OBJ_COMMIT: case GIT_OBJ_TREE: diff --git a/src/stash.c b/src/stash.c index 86e0a627c..22f756e35 100644 --- a/src/stash.c +++ b/src/stash.c @@ -634,7 +634,8 @@ int git_stash_drop( entry = git_reflog_entry_byindex(reflog, 0); git_reference_free(stash); - if ((error = git_reference_create(&stash, repo, GIT_REFS_STASH_FILE, &entry->oid_cur, 1, NULL, NULL) < 0)) + error = git_reference_create(&stash, repo, GIT_REFS_STASH_FILE, &entry->oid_cur, 1, NULL, NULL); + if (error < 0) goto cleanup; /* We need to undo the writing that we just did */ diff --git a/src/transports/smart_protocol.c b/src/transports/smart_protocol.c index 82891165f..5ca5065ac 100644 --- a/src/transports/smart_protocol.c +++ b/src/transports/smart_protocol.c @@ -314,7 +314,7 @@ static int wait_while_ack(gitno_buffer *buf) break; if (pkt->type == GIT_PKT_ACK && - (pkt->status != GIT_ACK_CONTINUE || + (pkt->status != GIT_ACK_CONTINUE && pkt->status != GIT_ACK_COMMON)) { git__free(pkt); return 0; |