Thread (11 messages) flat view 11 messages, 5 authors, 2016-08-11

Re: [PATCH] index-pack: correctly initialize appended objects

From: Björn Steinbrink <hidden>
Date: 2016-06-15 22:45:02

Possibly related (same subject, not in this thread)

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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help