| commit | b794ebeac9151af4a9136ef28a22a06c2afb17cc | |
| author | Jeff King <peff@peff.net> | |
| Sun, 13 Jul 2014 06:42:17 +0000 (13 02:42 -0400) | ||
| committer | Junio C Hamano <gitster@pobox.com> | |
| Mon, 28 Jul 2014 17:14:34 +0000 (28 10:14 -0700) | ||
| tree | 6ca634eb0048e97f868992642169b9e1cd99f387 | treesnapshot (tar.gz zip) |
| parent | 34dfe197a93763ec9c5d25eec499777cbb32b6fd | commitdiff |
diff-tree: avoid lookup_unknown_object
We generally want to avoid lookup_unknown_object, because it
results in allocating more memory for the object than may be
strictly necessary.
In this case, it is used to check whether we have an
already-parsed object before calling parse_object, to save
us from reading the object from disk. Using lookup_object
would be fine for that purpose, but we can take it a step
further. Since this code was written, parse_object already
learned the "check lookup_object" optimization, so we can
simply call parse_object directly.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
We generally want to avoid lookup_unknown_object, because it
results in allocating more memory for the object than may be
strictly necessary.
In this case, it is used to check whether we have an
already-parsed object before calling parse_object, to save
us from reading the object from disk. Using lookup_object
would be fine for that purpose, but we can take it a step
further. Since this code was written, parse_object already
learned the "check lookup_object" optimization, so we can
simply call parse_object directly.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| builtin/diff-tree.c | diffblobblamehistory |