Derrick Stolee <stolee@xxxxxxxxx> writes:
>> @@ -79,7 +79,7 @@ struct object_array {
>> * builtin/show-branch.c: 0-------------------------------------------26
>
> The only collision is here in builtin/show-branch.c, and when I added
> the TOPO_WALK_* bits, I didn't understand that these bits were
> sufficiently independent from the topo-walk logic that we could add
> collisions here.
The show-branch subcommand does its own walking without using any of
the usual revision machinery, and uses a bit per starting point (so
the number of starting points is limited by the wordsize), so it
should be safe.
It would be wonderful if the bits used by it can be moved to a slab
so that it can work from more starting points, or deprecate the
subcommand altogether. With better visualization tools (including
"log --graph --oneline"), I suspect that it outlived its usefulness.
