Merge branch 'jc/maint-unpack-objects-strict'
authorJunio C Hamano <gitster@pobox.com>
Thu, 27 Aug 2009 23:59:08 +0000 (16:59 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 27 Aug 2009 23:59:08 +0000 (16:59 -0700)
* jc/maint-unpack-objects-strict:
  Fix "unpack-objects --strict"

Conflicts:
builtin-unpack-objects.c

1  2 
builtin-unpack-objects.c

diff --combined builtin-unpack-objects.c
index 968fda7219807de9985c06eccd0812da04a149ae,bae00eabb0f8b712f2d737fe86cbd56421e0b441..685566e0b5e458c510fdf989744d63dda29e28f0
@@@ -68,7 -68,7 +68,7 @@@ static void *fill(int min
                if (ret <= 0) {
                        if (!ret)
                                die("early EOF");
 -                      die("read error on input: %s", strerror(errno));
 +                      die_errno("read error on input");
                }
                len += ret;
        } while (len < min);
@@@ -158,7 -158,7 +158,7 @@@ struct obj_info 
  #define FLAG_WRITTEN (1u<<21)
  
  static struct obj_info *obj_list;
 -unsigned nr_objects;
 +static unsigned nr_objects;
  
  /*
   * Called only from check_object() after it verified this object
@@@ -181,10 -181,10 +181,10 @@@ static void write_cached_object(struct 
  static int check_object(struct object *obj, int type, void *data)
  {
        if (!obj)
-               return 0;
+               return 1;
  
        if (obj->flags & FLAG_WRITTEN)
-               return 1;
+               return 0;
  
        if (type != OBJ_ANY && obj->type != type)
                die("object type mismatch");
                if (type != obj->type || type <= 0)
                        die("object of unexpected type");
                obj->flags |= FLAG_WRITTEN;
-               return 1;
+               return 0;
        }
  
        if (fsck_object(obj, 1, fsck_error_function))
                die("Error in object");
-       if (!fsck_walk(obj, check_object, NULL))
 -      if (fsck_walk(obj, check_object, 0))
++      if (fsck_walk(obj, check_object, NULL))
                die("Error on reachable objects of %s", sha1_to_hex(obj->sha1));
        write_cached_object(obj);
-       return 1;
+       return 0;
  }
  
  static void write_rest(void)
  {
        unsigned i;
-       for (i = 0; i < nr_objects; i++)
-               check_object(obj_list[i].obj, OBJ_ANY, NULL);
+       for (i = 0; i < nr_objects; i++) {
+               if (obj_list[i].obj)
 -                      check_object(obj_list[i].obj, OBJ_ANY, 0);
++                      check_object(obj_list[i].obj, OBJ_ANY, NULL);
+       }
  }
  
  static void added_object(unsigned nr, enum object_type type,
@@@ -495,8 -497,6 +497,8 @@@ int cmd_unpack_objects(int argc, const 
        int i;
        unsigned char sha1[20];
  
 +      read_replace_refs = 0;
 +
        git_config(git_default_config, NULL);
  
        quiet = !isatty(2);