apply: simplify build_fake_ancestor()
authorJunio C Hamano <gitster@pobox.com>
Fri, 1 Feb 2013 03:19:44 +0000 (19:19 -0800)
committerJunio C Hamano <gitster@pobox.com>
Fri, 1 Feb 2013 04:30:55 +0000 (20:30 -0800)
commite2afb0be9073fbe9f565ad9e952155c0bf7dd7c6
tree623ec8b09af697819d835d66701971ef4f276c94
parent4ae6d4699f4d1df7cd3ffac9dd361c86d62eca42
apply: simplify build_fake_ancestor()

The local variable sha1_ptr in the build_fake_ancestor() function
used to either point at the null_sha1[] (if the ancestor did not
have the path) or at sha1[] (if we read the object name into the
local array), but 7a98869 (apply: get rid of --index-info in favor
of --build-fake-ancestor, 2007-09-17) made the "missing in the
ancestor" case unnecessary, hence sha1_ptr, when used, always points
at the local array.

Get rid of the unneeded variable, and restructure the if/else
cascade a bit to make it easier to read.  There should be no
behaviour change.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/apply.c