author | Junio C Hamano <junkio@cox.net> | |
Sun, 19 Nov 2006 06:13:33 +0000 (22:13 -0800) | ||
committer | Junio C Hamano <junkio@cox.net> | |
Mon, 20 Nov 2006 02:44:29 +0000 (18:44 -0800) | ||
commit | ef06b91804ff5a626e265c6d9080bbc0e7924e88 | |
tree | a73de6571e4d96a0277ab2528a34cc993eb75bf6 | tree | snapshot |
parent | 6c96c0f1940b0247530d21b10042d16bf1e7b769 | commit | diff |
do_for_each_ref: perform the same sanity check for leftovers.
An earlier commit b37a562a added a check to see if the ref
points at a valid object (as a part of 'negative ref' support
which we currently do not use), but did so only while iterating
over both packed and loose refs, and forgot to apply the same
check while iterating over the remaining ones.
We might want to replace the "if null then omit it" check with
"eh --- what business does a 0{40} value have here?" complaint
later since we currently do not use negative refs, but that is
a separate issue.
Signed-off-by: Junio C Hamano <junkio@cox.net>
An earlier commit b37a562a added a check to see if the ref
points at a valid object (as a part of 'negative ref' support
which we currently do not use), but did so only while iterating
over both packed and loose refs, and forgot to apply the same
check while iterating over the remaining ones.
We might want to replace the "if null then omit it" check with
"eh --- what business does a 0{40} value have here?" complaint
later since we currently do not use negative refs, but that is
a separate issue.
Signed-off-by: Junio C Hamano <junkio@cox.net>
refs.c | diff | blob | history |