> 
> runTest: aSelector
> 
>       | actualResult expectedResult |
>       [ self
>               logTest: aSelector;
>               clearLog;
>               perform: aSelector ]
>                       on: MyTestError do:
>                               [ :ex | self log: 'Unhandled Exception'.
>                                       ex return: nil ].
> 
> as you can see, its a bit hard to quickly determine, where ends one
> block and starts another one.
> That's why i'm always putting ' ] ' at the next line, and align its
> indentation to be the same as in opening brace:
> 
>       [ self
>               logTest: aSelector;
>               clearLog;
>               perform: aSelector
>       ]
>       on: MyTestError do:
>               [ :ex | self log: 'Unhandled Exception'.
>                       ex return: nil
>               ].
> 
> do you agree that this one is much more readable

Not for me :)
May be I read too much Smalltalk code I prefer the rectangle approach of the 
first one ;) 
_______________________________________________
Pharo-project mailing list
[email protected]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project

Reply via email to