diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2022-11-21 20:26:55 +0100 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-11-22 02:13:54 +0100 |
commit | 40286ca2fa1e08c386ea7bc6b76616a3cac63ffd (patch) | |
tree | ce8f4f6cdab16cff74902780d997f5625a5ad226 /object.c | |
parent | parse_object(): check on-disk type of suspected blob (diff) | |
download | git-40286ca2fa1e08c386ea7bc6b76616a3cac63ffd.tar.xz git-40286ca2fa1e08c386ea7bc6b76616a3cac63ffd.zip |
parse_object(): simplify blob conditional
Commit 8db2dad7a0 (parse_object(): check on-disk type of suspected blob,
2022-11-17) simplified the conditional for checking if we might have a
blob. But we can simplify it further. In:
!obj || (obj && obj->type == OBJ_BLOB)
the short-circuit "OR" means "obj" will always be true on the right-hand
side. The compiler almost certainly optimized that out anyway, but
dropping it makes the conditional easier to understand for humans.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to '')
-rw-r--r-- | object.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -286,7 +286,7 @@ struct object *parse_object_with_flags(struct repository *r, return &commit->object; } - if ((!obj || (obj && obj->type == OBJ_BLOB)) && + if ((!obj || obj->type == OBJ_BLOB) && oid_object_info(r, oid, NULL) == OBJ_BLOB) { if (!skip_hash && stream_object_signature(r, repl) < 0) { error(_("hash mismatch %s"), oid_to_hex(oid)); |