diff options
| author | Alecs King <alecsk@gmail.com> | 2005-08-04 11:35:37 +0800 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2005-08-03 21:41:56 -0700 |
| commit | 635d37afff321a9c296ed5d6e0203f2a1c91373f (patch) | |
| tree | 74400f7c5508ef9cb635d8a9cab95dfe81747003 | |
| parent | 20f6633112c2a95b4907b4ff2e41e3c450395d34 (diff) | |
| download | git-635d37afff321a9c296ed5d6e0203f2a1c91373f.tar.gz git-635d37afff321a9c296ed5d6e0203f2a1c91373f.tar.xz | |
[PATCH] Fix sparse warnings
fix one 'should it be static?' warning and
two 'mixing declarations and code' warnings.
Signed-off-by: Alecs King <alecsk@gmail.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
| -rw-r--r-- | connect.c | 4 | ||||
| -rw-r--r-- | ssh-pull.c | 2 | ||||
| -rw-r--r-- | tools/mailinfo.c | 2 |
3 files changed, 3 insertions, 5 deletions
@@ -230,11 +230,9 @@ int match_refs(struct ref *src, struct ref *dst, struct ref ***dst_tail, if (src->peer_ref) continue; dst_peer = find_ref_by_name(dst, src->name); - if (dst_peer && dst_peer->peer_ref) + if ((dst_peer && dst_peer->peer_ref) || (!dst_peer && !all)) continue; if (!dst_peer) { - if (!all) - continue; /* Create a new one and link it */ int len = strlen(src->name) + 1; dst_peer = xcalloc(1, sizeof(*dst_peer) + len); diff --git a/ssh-pull.c b/ssh-pull.c index bdc99dfdc..b7f313bf2 100644 --- a/ssh-pull.c +++ b/ssh-pull.c @@ -10,7 +10,7 @@ static int fd_out; static unsigned char remote_version = 0; static unsigned char local_version = 1; -ssize_t force_write(int fd, void *buffer, size_t length) +static ssize_t force_write(int fd, void *buffer, size_t length) { ssize_t ret = 0; while (ret < length) { diff --git a/tools/mailinfo.c b/tools/mailinfo.c index 5a857f711..fb2ea2b70 100644 --- a/tools/mailinfo.c +++ b/tools/mailinfo.c @@ -152,13 +152,13 @@ static void cleanup_space(char *buf) static void handle_rest(void) { + FILE *out = cmitmsg; char *sub = cleanup_subject(subject); cleanup_space(name); cleanup_space(date); cleanup_space(email); cleanup_space(sub); printf("Author: %s\nEmail: %s\nSubject: %s\nDate: %s\n\n", name, email, sub, date); - FILE *out = cmitmsg; do { if (!memcmp("diff -", line, 6) || |