Code

tree_entry_interesting(): remove dependency on struct diff_options
authorNguyễn Thái Ngọc Duy <pclouds@gmail.com>
Wed, 15 Dec 2010 15:02:39 +0000 (22:02 +0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 16 Dec 2010 21:14:24 +0000 (13:14 -0800)
commita79ca12d0a0cf4f731d26a6a0bcd1ad4eaf5f0ff
tree008df2d22dabab29e6bdfe597374ee91ab2d2a4e
parente13219fa1a3769cff74ce36acef7a8306795ed52
tree_entry_interesting(): remove dependency on struct diff_options

This function can be potentially used in more places than just
tree-diff.c. "struct diff_options" does not make much sense outside
diff_tree_sha1().

While removing the use of diff_options, it also removes
tree_entry_extract() call, which means S_ISDIR() uses the entry->mode
directly, without being filtered by canon_mode() (called internally
inside tree_entry_extract).

The only use of the mode information in this function is to check the
type of the entry by giving it to S_ISDIR() macro, and the result does
not change with or without canon_mode(), so it is ok to bypass
tree_entry_extract().

Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
tree-diff.c