On 2008.07.24 22:21:14 -0700, Junio C Hamano wrote: > Reading get_data_from_pack(), it does rely on hdr_size, idx.offset and > idx.offset of the next entry to be set correctly. The function does not > seem to use type (which the patch is also setting) nor real_type (which > the patch does not set). type is used in get_base_data(). > However, the code checks objects[nth].real_type all over the place in the > code. Doesn't the lack of real_type assignment in append_obj_to_pack() > affect them in any way? I had thought that resolve_delta() would set that, but it seems that we never call that function like that. Hm... Björn -- 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