Jonathan Tan wrote:

Jonathan Nieder wrote:

quoted

Hm, where does this dichotomy come from?  E.g. is the latter a
lower-level function used by the former?

I don't know the reason for the dichotomy - perhaps it was an oversight.

Ah, that makes sense.

[...]

This might be a moot point, but what do you mean by the
"'has_object_file || find_cached_object' pattern"?

Oh!  Thanks, I should have been more precise.  And calling it a pattern
was probably overreaching --- I can only find one instance, in
pretend_object_file:

	if (has_object_file(oid) || find_cached_object(oid))
		return 0;

Since there's only one, I was on the wrong track.  (Except that with
this change, that can indeed change to

	if (has_object_file(oid))
		return 0;

Thanks,
Jonathan