On Wed, Aug 8, 2018 at 4:25 PM, Jeff King <peff@xxxxxxxx> wrote: > Even if you just use the oid to do a separate lookup in the object > database, there's still a benefit in accessing the objects in pack > order. You're probably right, but I don't immediately see what the benefit is. On a not completely unrelated note, I just realized that in my patch, i should be pack_nr (ordinal in pack order) and pack_nr should be index_nr (ordinal in index order, i.e. alphabetical order). If you run with this, feel free to write your own patch and maybe I'll learn how accessing objects in pack order benefits looking up the object database through the commit message. > I can try to pick this up and carry the cat-file bits to completion if > you want, but probably not until tomorrow or Friday. Thanks - feel free to do this.