> [1/4]: bitmap_has_sha1_in_uninteresting(): drop BUG check > > The actual fix. This should get merged to next ASAP (or the original > topic just reverted). > > [2/4]: t5310: test delta reuse with bitmaps > > I did this separately to give us flexibility to squash or merge > quickly. But it does find Ævar's bug on a git without patch 1. > > [3/4]: traverse_bitmap_commit_list(): don't free result > > The original assert should have simply been useless, but it was the > surprising behavior of this function that turned it into a bug. > > [4/4]: pack-bitmap: drop "loaded" flag > > And this is just an annoyance I ran into, which is a fallout from > our conversion to using an allocated bitmap_index struct.
FWIW, the whole series is Reviewed-by: Stefan Beller <sbel...@google.com> I am sorry for suggesting the BUG() in the first place. Stefan
