[PATCH v3 5/6] cache_tree_find(): remove early return

From: Michael Haggerty <mhagger@alum.mit.edu>
Date: Mar 5, 2014, 17:26 UTC
Message-ID: <1394040390-7954-6-git-send-email-mhagger@alum.mit.edu>
In-Reply-To: <1394040390-7954-1-git-send-email-mhagger@alum.mit.edu>

There is no need for an early

    return it;

from the loop if slash points at the end of the string, because that is exactly what will happen when the while condition fails at the start of the next iteration.

Signed-off-by: Michael Haggerty <mhagger@alum.mit.edu>
---
 cache-tree.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/cache-tree.c b/cache-tree.c
index 39ad8c9..17db9f9 100644
--- a/cache-tree.c
+++ b/cache-tree.c
@@ -565,8 +565,6 @@ static struct cache_tree *cache_tree_find(struct cache_tree *it, const char *pat
 		it = sub->cache_tree;
 		while (*slash == '/')
 			slash++;
-		if (!*slash)
-			return it; /* prefix ended with slashes */
 		path = slash;
 	}
 	return it;

-- 
1.9.0