refs API: make loose_fill_ref_dir() not set errno
Change the refs_resolve_ref_unsafe() invoked in loose_fill_ref_dir() to a form that ignores errno. The only eventual caller of this function is create_ref_cache(), whose callers in turn don't have their failure depend on any errno set here. Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>maint
parent
db7a3d25d6
commit
096a7fbb97
|
@ -280,10 +280,11 @@ static void loose_fill_ref_dir(struct ref_store *ref_store,
|
||||||
create_dir_entry(dir->cache, refname.buf,
|
create_dir_entry(dir->cache, refname.buf,
|
||||||
refname.len));
|
refname.len));
|
||||||
} else {
|
} else {
|
||||||
if (!refs_resolve_ref_unsafe(&refs->base,
|
int ignore_errno;
|
||||||
|
if (!refs_werrres_ref_unsafe(&refs->base,
|
||||||
refname.buf,
|
refname.buf,
|
||||||
RESOLVE_REF_READING,
|
RESOLVE_REF_READING,
|
||||||
&oid, &flag)) {
|
&oid, &flag, &ignore_errno)) {
|
||||||
oidclr(&oid);
|
oidclr(&oid);
|
||||||
flag |= REF_ISBROKEN;
|
flag |= REF_ISBROKEN;
|
||||||
} else if (is_null_oid(&oid)) {
|
} else if (is_null_oid(&oid)) {
|
||||||
|
|
Loading…
Reference in New Issue