Re: [PATCH] cache_tree_find(): remove redundant check in while condition

2014-03-03 Thread David Kastrup
Michael Haggerty writes: > On 03/03/2014 05:32 PM, David Kastrup wrote: >> [...] >> So perhaps all of that should just be >> >> while (*slash == '/') >> slash++; >> if (!*slash) >> return it; >> > > I just fixed a little thing

Re: [PATCH] cache_tree_find(): remove redundant check in while condition

2014-03-03 Thread Michael Haggerty
On 03/03/2014 05:32 PM, David Kastrup wrote: > [...] > So perhaps all of that should just be > > while (*slash == '/') > slash++; > if (!*slash) > return it; > I just fixed a little thing that caught my eye. You OWNED it.

Re: [PATCH] cache_tree_find(): remove redundant check in while condition

2014-03-03 Thread David Kastrup
Michael Haggerty writes: > Signed-off-by: Michael Haggerty > --- > A trivial little cleanup. > > cache-tree.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/cache-tree.c b/cache-tree.c > index 0bbec43..7f63c23 100644 > --- a/cache-tree.c > +++ b/cache-tree.c > @@ -565,

[PATCH] cache_tree_find(): remove redundant check in while condition

2014-03-03 Thread Michael Haggerty
Signed-off-by: Michael Haggerty --- A trivial little cleanup. cache-tree.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cache-tree.c b/cache-tree.c index 0bbec43..7f63c23 100644 --- a/cache-tree.c +++ b/cache-tree.c @@ -565,7 +565,7 @@ static struct cache_tree *cache_tree_