A perfect example of careless programming.
"I'll do it my way, and if it causing any problems, i don't care and i will
just ignore them. And it's not my problem anyways, i went to something else
already, since this part is works and DONE"
:)

On 30 March 2016 at 14:33, Nicolai Hess <[email protected]> wrote:

> Please don't do this:
>
> updateHeight
>     "no need to care about height, when it's logic is not customized"
>     self layout isHeightCustom ifFalse: [ ^ self ].
>     [ self bounds: (self brickBounds withHeight: self customHeight) ]
>         on: Exception
>         do: [ "just skip and do nothing" ]
>
> This makes debugging GLM/Brick ui/layout code with "self haltOnce"
> impossible.
> see
> GLMBrickGeometryTrait>>#updateHeight
> GLMBrickGeometryTrait>>#updateWidth
>
> And if you log out the raised exception, you see some calls to
> not initialized fonts and a ZeroDevide and some more errors.
> The above catch, catches and hides wrong / to late initialized objects.
>



-- 
Best regards,
Igor Stasenko.

Reply via email to