Code

Use "-Wall -O2" for the compiler to get more warnings.
authorLinus Torvalds <torvalds@ppc970.osdl.org>
Fri, 8 Apr 2005 16:59:28 +0000 (09:59 -0700)
committerLinus Torvalds <torvalds@ppc970.osdl.org>
Fri, 8 Apr 2005 16:59:28 +0000 (09:59 -0700)
And fix up the warnings that it pointed out. Let's keep the tree
clean from early on.

Not that the code is very beautiful anyway ;)

Makefile
cat-file.c
init-db.c
read-cache.c
read-tree.c
show-diff.c
update-cache.c
write-tree.c

index a6bba79ba1f46a1bbf7773449c3bd2bb9bf48e8b..6a0a1c958fa2a1a7020c958e1f7337bd3b86a9f8 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -1,4 +1,4 @@
-CFLAGS=-g
+CFLAGS=-g -O3 -Wall
 CC=gcc
 
 PROG=update-cache show-diff init-db write-tree read-tree commit-tree cat-file
index 3071672c6ce2ffb7c1c7d43c1a728f8e020e296b..3829fb6097ad9840cca7654f0b8a718100d590ac 100644 (file)
@@ -11,7 +11,6 @@ int main(int argc, char **argv)
        char type[20];
        void *buf;
        unsigned long size;
-       int fd;
 
        if (argc != 3 || get_sha1_hex(argv[2], sha1))
                usage("cat-file: cat-file [-t | tagname] <sha1>");
index bc65086368b44f37a8c7e47734e5b0dffed8873c..1fcef0b373ccec8693ffab22adfe94f4f3c5b86d 100644 (file)
--- a/init-db.c
+++ b/init-db.c
@@ -8,7 +8,7 @@
 int main(int argc, char **argv)
 {
        char *sha1_dir = getenv(DB_ENVIRONMENT), *path;
-       int len, i, fd;
+       int len, i;
 
        if (mkdir(".dircache", 0700) < 0) {
                perror("unable to create .dircache");
@@ -25,7 +25,7 @@ int main(int argc, char **argv)
        if (sha1_dir) {
                struct stat st;
                if (!stat(sha1_dir, &st) < 0 && S_ISDIR(st.st_mode))
-                       return;
+                       return 0;
                fprintf(stderr, "DB_ENVIRONMENT set to bad directory %s: ", sha1_dir);
        }
 
index b151981c1b3d15da75411c2af856958d06138c30..60a0b5b2e78f403047f3a5e2cac1c5c7965091a1 100644 (file)
@@ -89,7 +89,7 @@ void * read_sha1_file(unsigned char *sha1, char *type, unsigned long *size)
        z_stream stream;
        char buffer[8192];
        struct stat st;
-       int i, fd, ret, bytes;
+       int fd, ret, bytes;
        void *map, *buf;
        char *filename = sha1_file_name(sha1);
 
@@ -173,7 +173,7 @@ int write_sha1_file(char *buf, unsigned len)
 int write_sha1_buffer(unsigned char *sha1, void *buf, unsigned int size)
 {
        char *filename = sha1_file_name(sha1);
-       int i, fd;
+       int fd;
 
        fd = open(filename, O_WRONLY | O_CREAT | O_EXCL, 0666);
        if (fd < 0)
@@ -228,6 +228,7 @@ int read_cache(void)
        if (fd < 0)
                return (errno == ENOENT) ? 0 : error("open failed");
 
+       size = 0; // avoid gcc warning
        map = (void *)-1;
        if (!fstat(fd, &st)) {
                map = NULL;
index 3785e259756b4b1f396baabcff07a9bcb6f75ddc..af486ff33f6f66323b25efc5ce9e3540d5f5bab9 100644 (file)
@@ -72,7 +72,6 @@ static int unpack(unsigned char *sha1)
 
 int main(int argc, char **argv)
 {
-       int fd;
        unsigned char sha1[20];
 
        if (argc != 2)
index e7aa60b4ab92ab2d0fe6afaeb5657757d0401225..3c6d4f98b48634d63554821b8d23592e359f8c91 100644 (file)
@@ -60,7 +60,6 @@ int main(int argc, char **argv)
                struct stat st;
                struct cache_entry *ce = active_cache[i];
                int n, changed;
-               unsigned int mode;
                unsigned long size;
                char type[20];
                void *new;
index 0bcd5d408191333e70f54fbf28676a7e3696ced5..413e09d48fa0aad26ef4863a47e1c75c08844d81 100644 (file)
@@ -50,6 +50,7 @@ static int remove_file_from_cache(char *path)
                if (pos < active_nr)
                        memmove(active_cache + pos, active_cache + pos + 1, (active_nr - pos - 1) * sizeof(struct cache_entry *));
        }
+       return 0;
 }
 
 static int add_cache_entry(struct cache_entry *ce)
@@ -250,4 +251,5 @@ int main(int argc, char **argv)
                return 0;
 out:
        unlink(".dircache/index.lock");
+       return 0;
 }
index 012eeeff1b5dd90149ebcd9eed139698c9fab035..6e531b40a84914cf1b37e12f8629fecbec0ae041 100644 (file)
@@ -31,7 +31,7 @@ static int prepend_integer(char *buffer, unsigned val, int i)
 
 int main(int argc, char **argv)
 {
-       unsigned long size, offset, val;
+       unsigned long size, offset;
        int i, entries = read_cache();
        char *buffer;