From aa16dd0b1742ad317615d099aa6301a6c5b32974 Mon Sep 17 00:00:00 2001 From: Runxi Yu Date: Sun, 16 Nov 2025 00:00:00 +0000 Subject: EntryRecursive should return ErrNotFound instead of nil, nil --- obj_tree.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'obj_tree.go') diff --git a/obj_tree.go b/obj_tree.go index d4246285..06d88692 100644 --- a/obj_tree.go +++ b/obj_tree.go @@ -148,7 +148,6 @@ func (tree *Tree) Entry(name []byte) *TreeEntry { // EntryRecursive looks up a tree entry by path. // // Lookups are recursive. -// It returns nil if no such entry exists. func (sTree *StoredTree) EntryRecursive(repo *Repository, path [][]byte) (*TreeEntry, error) { if len(path) == 0 { return nil, errors.New("furgit: tree: empty path") @@ -158,7 +157,7 @@ func (sTree *StoredTree) EntryRecursive(repo *Repository, path [][]byte) (*TreeE for i, part := range path { entry := currentTree.Entry(part) if entry == nil { - return nil, nil + return nil, ErrNotFound } if i == len(path)-1 { return entry, nil @@ -176,5 +175,5 @@ func (sTree *StoredTree) EntryRecursive(repo *Repository, path [][]byte) (*TreeE currentTree = nextTree } - return nil, nil + return nil, ErrNotFound } -- cgit v1.3.1-10-gc9f91