2016-05-06 10:26 GMT+02:00 Peter Uhnák <i.uh...@gmail.com>:
> 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.
>
would this help (added to the very start of needsParenthesisFor: , after
checking that parent isn't nil) :
parent isArray ifTrue: [ ^ false ].
>
> 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
>
>
>
