Skip to content

Commit

Permalink
tree-diff: simplify tree_entry_pathcmp
Browse files Browse the repository at this point in the history
Since an earlier "Finally switch over tree descriptors to contain a
pre-parsed entry", we can safely access all tree_desc->entry fields
directly instead of first "extracting" them through
tree_entry_extract.

Use it. The code generated stays the same - only it now visually looks
cleaner.

Signed-off-by: Kirill Smelkov <kirr@mns.spb.ru>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Kirill Smelkov authored and Junio C Hamano committed Mar 20, 2014
1 parent 5acabd8 commit 1a27a15
Showing 1 changed file with 6 additions and 11 deletions.
17 changes: 6 additions & 11 deletions tree-diff.c
Original file line number Diff line number Diff line change
Expand Up @@ -15,18 +15,13 @@
*/
static int tree_entry_pathcmp(struct tree_desc *t1, struct tree_desc *t2)
{
unsigned mode1, mode2;
const char *path1, *path2;
const unsigned char *sha1, *sha2;
int cmp, pathlen1, pathlen2;
struct name_entry *e1, *e2;
int cmp;

sha1 = tree_entry_extract(t1, &path1, &mode1);
sha2 = tree_entry_extract(t2, &path2, &mode2);

pathlen1 = tree_entry_len(&t1->entry);
pathlen2 = tree_entry_len(&t2->entry);

cmp = base_name_compare(path1, pathlen1, mode1, path2, pathlen2, mode2);
e1 = &t1->entry;
e2 = &t2->entry;
cmp = base_name_compare(e1->path, tree_entry_len(e1), e1->mode,
e2->path, tree_entry_len(e2), e2->mode);
return cmp;
}

Expand Down

0 comments on commit 1a27a15

Please sign in to comment.