Nov. 13, 2019
9:05 a.m.
On 13 Nov 2019, at 09:04, ducasse <stepharo@netcourrier.com> wrote:
Hi
In the new implementation of the code formattter I will remove useTraditionalPrecedence. This settings put additional parentheses around binary selector but I find it cumbersome
While it may make sense for
it turns foo ^ 1 + 2 * 3
into
foo ^ (1 + 2) * 3
foo ^ 1 * 3 + 2
foo ^ (1 | 2) % 3
foo
^ 1 % 2 | 3
So let me know what you think but I will be hard to convince :)
I vote to remove it. The RB AST encodes parenthesis information, so that means that we can have a pretty printer keeping the parenthesis if they have been added by the original author. Adding new ones makes no sense to me. So I vote to remove. Marcus