On Fri, Nov 01, 2024 at 08:16:06AM -0400, Jeff King wrote: > builtin/rev-list.c | 7 +++++++ > t/t5310-pack-bitmaps.sh | 12 ++++++++++++ > 2 files changed, 19 insertions(+) Nice find, and well explained (not just why it doesn't work today but could in the future, but why making it work in the future with bitmaps does not necessarily a clear performance improvement). Probably you and I should think more about other rev-list options that *don't* work with --use-bitmap-index. I share your feeling there that there probably exist such options which silently do nothing (or the wrong thing) in the presence of '--use-bitmap-index'. Let's merge this one down. Thanks, Taylor
