Sorry for the late reply.
> Let's make the behavior consistent with modified files by showing
> all-zero values also for intent-to-add files.
>
-- >8 --
>
> diff_addremove(&revs->diffopt, '+', ce->ce_mode,
> - the_hash_algo->empty_tree, 0,
> - ce->name, 0);
> + &null_oid, 0, ce->name, 0);
> continue;
> }
>
> diff --git a/t/t2203-add-intent.sh b/t/t2203-add-intent.sh
> index 8a5d55054f..cf0175ad6e 100755
> --- a/t/t2203-add-intent.sh
> +++ b/t/t2203-add-intent.sh
> @@ -240,7 +240,6 @@ test_expect_success 'i-t-a files shown as new for "diff", "diff-files"; not-new
>
> hash_e=$(git hash-object empty) &&
> hash_n=$(git hash-object not-empty) &&
> - hash_t=$(git hash-object -t tree /dev/null) &&
>
> cat >expect.diff_p <<-EOF &&
> diff --git a/empty b/empty
> @@ -259,8 +258,8 @@ test_expect_success 'i-t-a files shown as new for "diff", "diff-files"; not-new
> create mode 100644 not-empty
> EOF
> cat >expect.diff_a <<-EOF &&
> - :000000 100644 0000000 $(git rev-parse --short $hash_t) A$(printf "\t")empty
> - :000000 100644 0000000 $(git rev-parse --short $hash_t) A$(printf "\t")not-empty
> + :000000 100644 0000000 0000000 A$(printf "\t")empty
> + :000000 100644 0000000 0000000 A$(printf "\t")not-empty
This change (and the new test in [PATCH v4 2/2]) looks good to me.
I learnt quite a bit about what the string "0{40}" means in the context
of `diff' post-image from the conversations around this patch. It was
very helpful.
Thank you.
