I have seen this too, and I hate it as well. I would definitively prefer the minimal syntax: no parenthesis when not needed, not extra $# when not needed.
On 06 May 2016, at 10:26, Peter Uhnák <i.uhnak@gmail.com> wrote:
Hi,
I thought that I already asked about this but apparently not...
Right now when I have an array like
observations := { MElement -> { #change -> #(name owner uuid). #add -> #(). #remove -> #() }.
MContainer -> { #change -> #(). #add -> #(add:). #remove -> #(remove:). }. }.
it gets formatted into
observations := {(MElement -> {(#change -> #(#name #owner #uuid)). (#add -> #()). (#remove -> #())}). (MContainer -> {(#change -> #()). (#add -> #(#add:)). (#remove -> #(#remove:))})}.
Which is really ugly with all the _uneeded_ parenthesis.
I tried to look into BlConfigurableFormatter>>needsParenthesisFor: where it is being added, but any change I try to make I end up breaking some test as the behavior is not very obvious for me.
Can this be changed/fixed easily?
And second, perhaps less important... do we really need to reformat #(name owner uuid) into #(#name #owner #uuid)? Seems a bit superfluous too.
Thanks, Peter