On Wed, Oct 18, 2023 at 04:18:23PM -0700, Junio C Hamano wrote: > Taylor Blau <me@xxxxxxxxxxxx> writes: > > > Now that we have factored out many of the common routines necessary to > > index a new object into a pack created by the bulk-checkin machinery, we > > can introduce a variant of `index_blob_bulk_checkin()` that acts on > > blobs whose contents we can fit in memory. > > Hmph. > > Doesn't the duplication between the main loop of the new > deflate_obj_contents_to_pack() with existing deflate_blob_to_pack() > bother you? Yeah, I am not sure how I missed seeing the opportunity to clean that up. Another (hopefully final...) reroll incoming. Thanks, Taylor