Hi!

We were browsing with Steph users of #announce: and we arrived to the following method:

GLMPresentation >> window
    self announce: (GLMWindowRequest new
        action: [ :aWindow | ^ aWindow ]).
    ^ nil

And we could not stop noticing that, while the announcement, probably originally meant to decouple the Presentation and the window, is now coupling them in a non-clear way... I mean, the coupling is there, but is now hidden in an announcement.

Also, the usage of the non-local return makes the assumption that the announcement will be executed on the same process... And that is at most strange, but also looks buggy.

Why is this method not a normal accessor?

Guille

Reply via email to