On Wed, Dec 14, 2022 at 11:17:39AM -0800, Jonathan Tan wrote: > Thanks everyone once again and sorry for the churn. Hopefully I got it > right this time. > > open_loose_object() is documented to return the path of the object > we found, so I think we already have that covered (if we detect that > an object is corrupt, it follows that we would already have found the > object in the first place). This version looks good to me. Thanks for your persistence. :) I think the end result is very nicely done. -Peff