On Fri, 27 Oct 2006, Junio C Hamano wrote:
> Nicolas Pitre <nico@xxxxxxx> writes:
>
> > +static void fix_unresolved_deltas(int nr_unresolved)
> > +{
[...]
> > + if (objects[d->obj_no].real_type != OBJ_REF_DELTA)
> > + continue;
>
> Just for the sake of my own sanity checking,...
>
> Is it because base objects of OFS_DELTA _must_ exist in the same
> pack (even when --thin) that we do not have to worry about them
> in this function?
Indeed. Otherwise it would be impossible to determine the offset to the
base object if that base object is not in the pack.
That doesn't mean there isn't any OFS_DELTA left to resolve though.
But this function is only about adding missing objects to the pack and
the deltas that need them must be REF_DELTA objects.
Once a REF_DELTA has been resolved with an external object, it then
constitute a possible new base for remaining OFS_DELTA or
REF_DELTA objects, and that is found recursively in resolve_delta().
Nicolas
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html