Re: [PATCH v9 0/4] graph: indent visual roots in graph

> I agree that having an infinite stair is not a good solution. the 3
> column wrap looks reasonable.
>
> I see two cases with this wrap:
>
> 1. No conflict case:
>
>   A
>     B
>       C
>   D
>     E
>       F
>
> No ambiguity, this would be the ideal case.
>
> 2. Ambiguity:
>
> If it happens that the visual number on visual roots meet the condition
> (number_of_visual_roots % 3 == 0) and the next commit is NOT a visual
> root this would happen:
>
>   A
>     B
>       C
>   D
>   E
>   E
>
> Which would be ambiguous. The solution is to check with the lookahead
> buffer that we have since patch 3 if the next is a visual root, if it's
> not we indent D anyway:
>
>   A
>     B
>       C
>     D
>   E
>   E
>
> Which I find the pyramid effect uncomfortable.
> What about capping at 4 columns?
>
> 1.
>
>   A
>     B
>       C
>         D
>   E
>     F
>       G
>         H
>
> 2.
>
>   A
>     B
>       C
>         D
>     E
>   F
>   F
>
> I prefer the 4 column wrap because it looks more abrupt and IMO shows
> better that the commits are unrelated.
>
> What do you think?