Jeff King <p...@peff.net> writes:
> As a general rule (and why I'm raising this issue in reply to Jonathan's
> patch), I think most or all sites that want OBJECT_INFO_QUICK will want
> SKIP_FETCH_OBJECT as well, and vice versa. The reasoning is generally
> the same:
>
> - it's OK to racily have a false negative (we'll still be correct, but
> possibly a little less optimal)
>
> - it's expected and normal to be missing the object, so spending time
> double-checking the pack store wastes measurable time in real-world
> cases
31f5256c ("sha1-file: split OBJECT_INFO_FOR_PREFETCH", 2019-05-28)
separated SKIP_FETCH_OBJECT out of FOR_PREFETCH, the latter of which
was and is SKIP_FETCH and QUICK combined. Use SKIP_FETCH_OBJECT
alone may need to be re-examined and discouraged?
