Merge the new object model thing from Daniel Barkalow

[git.git] / fsck-cache.c

diff --git a/fsck-cache.c b/fsck-cache.c
index b8b66d7..edaf9e4 100644 (file)
--- a/fsck-cache.c
+++ b/fsck-cache.c
@@ -3,7 +3,11 @@
 #include <sys/types.h>
 #include <dirent.h>
 
-#include "revision.h"
+#include "commit.h"
+#include "tree.h"
+#include "blob.h"
+
+#define REACHABLE 0x0001
 
 static int show_unreachable = 0;
 static unsigned char head_sha1[20];
@@ -13,96 +17,54 @@ static void check_connectivity(void)
        int i;
 
        /* Look up all the requirements, warn about missing objects.. */
-       for (i = 0; i < nr_revs; i++) {
-               struct revision *rev = revs[i];
+       for (i = 0; i < nr_objs; i++) {
+               struct object *obj = objs[i];
 
-               if (show_unreachable && !(rev->flags & REACHABLE)) {
-                       printf("unreachable %s %s\n", rev->tag, sha1_to_hex(rev->sha1));
+               if (show_unreachable && !(obj->flags & REACHABLE)) {
+                       printf("unreachable %s\n", sha1_to_hex(obj->sha1));
                        continue;
                }
 
-               switch (rev->flags & (SEEN | USED)) {
-               case 0:
-                       printf("bad %s %s\n", rev->tag, sha1_to_hex(rev->sha1));
-                       break;
-               case USED:
-                       printf("missing %s, %s\n", rev->tag, sha1_to_hex(rev->sha1));
-                       break;
-               case SEEN:
-                       printf("dangling %s %s\n", rev->tag, sha1_to_hex(rev->sha1));
-                       break;
+               if (!obj->parsed) {
+                       printf("missing %s %s\n", obj->type, 
+                              sha1_to_hex(obj->sha1));
+               }
+               if (!obj->used) {
+                       printf("dangling %s %s\n", obj->type, 
+                              sha1_to_hex(obj->sha1));
                }
        }
 }
 
-static void mark_needs_sha1(unsigned char *parent, const char *ptag, unsigned char *child, const char *ctag)
-{
-       struct revision * child_rev = add_relationship(lookup_rev(parent, ptag), child, ctag);
-       child_rev->flags |= USED;
-}
-
-static int mark_sha1_seen(unsigned char *sha1, const char *tag)
-{
-       struct revision *rev = lookup_rev(sha1, tag);
-
-       rev->flags |= SEEN;
-       return 0;
-}
-
 static int fsck_tree(unsigned char *sha1, void *data, unsigned long size)
 {
-       int warn_old_tree = 1;
-
-       while (size) {
-               int len = 1+strlen(data);
-               unsigned char *file_sha1 = data + len;
-               char *path = strchr(data, ' ');
-               unsigned int mode;
-               if (size < len + 20 || !path || sscanf(data, "%o", &mode) != 1)
-                       return -1;
-
-               /* Warn about trees that don't do the recursive thing.. */
-               if (warn_old_tree && strchr(path, '/')) {
-                       fprintf(stderr, "warning: fsck-cache: tree %s has full pathnames in it\n", sha1_to_hex(sha1));
-                       warn_old_tree = 0;
-               }
-
-               data += len + 20;
-               size -= len + 20;
-               mark_needs_sha1(sha1, "tree", file_sha1, S_ISDIR(mode) ? "tree" : "blob");
+       struct tree *item = lookup_tree(sha1);
+       if (parse_tree(item))
+               return -1;
+       if (item->has_full_path) {
+               fprintf(stderr, "warning: fsck-cache: tree %s "
+                       "has full pathnames in it\n", sha1_to_hex(sha1));
        }
        return 0;
 }
 
 static int fsck_commit(unsigned char *sha1, void *data, unsigned long size)
 {
-       int parents;
-       unsigned char tree_sha1[20];
-       unsigned char parent_sha1[20];
-
-       if (memcmp(data, "tree ", 5))
+       struct commit *commit = lookup_commit(sha1);
+       if (parse_commit(commit))
                return -1;
-       if (get_sha1_hex(data + 5, tree_sha1) < 0)
+       if (!commit->tree)
                return -1;
-       mark_needs_sha1(sha1, "commit", tree_sha1, "tree");
-       data += 5 + 40 + 1;     /* "tree " + <hex sha1> + '\n' */
-       parents = 0;
-       while (!memcmp(data, "parent ", 7)) {
-               if (get_sha1_hex(data + 7, parent_sha1) < 0)
-                       return -1;
-               mark_needs_sha1(sha1, "commit", parent_sha1, "commit");
-               data += 7 + 40 + 1;     /* "parent " + <hex sha1> + '\n' */
-               parents++;
-       }
-       if (!parents)
+       if (!commit->parents)
                printf("root %s\n", sha1_to_hex(sha1));
        return 0;
 }
 
-static int fsck_entry(unsigned char *sha1, char *tag, void *data, unsigned long size)
+static int fsck_entry(unsigned char *sha1, char *tag, void *data, 
+                     unsigned long size)
 {
        if (!strcmp(tag, "blob")) {
-               /* Nothing to check */;
+               lookup_blob(sha1); /* Nothing to check; but notice it. */
        } else if (!strcmp(tag, "tree")) {
                if (fsck_tree(sha1, data, size) < 0)
                        return -1;
@@ -111,7 +73,7 @@ static int fsck_entry(unsigned char *sha1, char *tag, void *data, unsigned long
                        return -1;
        } else
                return -1;
-       return mark_sha1_seen(sha1, tag);
+       return 0;
 }
 
 static int fsck_name(char *hex)
@@ -125,7 +87,8 @@ static int fsck_name(char *hex)
                        unsigned long size;
                        void *buffer = NULL;
                        if (!check_sha1_signature(sha1, map, mapsize))
-                               buffer = unpack_sha1_file(map, mapsize, type, &size);
+                               buffer = unpack_sha1_file(map, mapsize, type,
+                                                         &size);
                        munmap(map, mapsize);
                        if (buffer && !fsck_entry(sha1, type, buffer, size))
                                return 0;
@@ -186,7 +149,9 @@ int main(int argc, char **argv)
                        continue;
                }
                if (!get_sha1_hex(argv[i], head_sha1)) {
-                       mark_reachable(lookup_rev(head_sha1, "commit"), REACHABLE);
+                       struct object *obj = &lookup_commit(head_sha1)->object;
+                       obj->used = 1;
+                       mark_reachable(obj, REACHABLE);
                        heads++;
                        continue;
                }