[Pharo-dev] deprecating use traditional formatter

Marcus Denker marcus.denker at inria.fr
Wed Nov 13 04:05:06 EST 2019



> On 13 Nov 2019, at 09:04, ducasse <stepharo at 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




More information about the Pharo-dev mailing list