Anders Kaseorg wrote: > Yeah. Okay, I see that âlookupâ implies unpacking, as opposed to âfindâ, > which doesnât. The important part is to avoid the find, of course, > because the I/O is expensive. > > Anyway, hereâs a series with that change split out. Thanks. In theory finding an object would only require looking at the pack index (and directory listings), but find_pack_entry does /* * We are about to tell the caller where they can * locate the requested object. We better make * sure the packfile is still here and can be * accessed before supplying that answer, as * it may have been deleted since the index * was loaded! */ which spoils that plan. Your original use of the word "lookup" seemed sane, too; I was just fishing for a more detailed explanation. Reviewed-by: Jonathan Nieder <jrnieder@xxxxxxxxx> > -- 8< -- > From 2ad1e58b8f6e9c117c77748b6e8b85227d9d5412 Mon Sep 17 00:00:00 2001 [...] For the future: this does not follow the preferred patch format (one patch per message, each of the form "description/---/diff"). In this case I am guessing it won't be much trouble to massage on the receiving end, so no terrible loss. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html