As described in previous commit, when recursing into sub-trees, we can
use lower-level tree walker, since its interface is now sha1 based.

The change is ok, because diff_tree_sha1() only invokes
__diff_tree_sha1(), and also, if base is empty, try_to_follow_renames().
But base is not empty here, as we have added a path and '/' before
recursing.

Signed-off-by: Kirill Smelkov <k...@mns.spb.ru>
---
 tree-diff.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tree-diff.c b/tree-diff.c
index dd6c760..e385ed4 100644
--- a/tree-diff.c
+++ b/tree-diff.c
@@ -116,8 +116,8 @@ static void show_path(struct strbuf *base, struct 
diff_options *opt,
 
        if (recurse) {
                strbuf_addch(base, '/');
-               diff_tree_sha1(t1 ? t1->entry.sha1 : NULL,
-                              t2 ? t2->entry.sha1 : NULL, base->buf, opt);
+               __diff_tree_sha1(t1 ? t1->entry.sha1 : NULL,
+                                t2 ? t2->entry.sha1 : NULL, base->buf, opt);
        }
 
        strbuf_setlen(base, old_baselen);
-- 
1.9.rc1.181.g641f458

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to