Feedback on issue 1703

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Feedback on issue 1703

Mariano Martinez Peck
Does someone has something to comment for http://code.google.com/p/pharo/issues/detail?id=1703  ?

Thanks

Mariano

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: Feedback on issue 1703

Stéphane Ducasse
Issue 1703: Closing last windows raise error
1 person starred this issue and may be notified of changes. Back to list
Status: Closed
Owner: marianopeck
Closed: Today
Cc: mike.rueger
Milestone-1.1


Add a comment andmake changes below
 
Reported by marianopeck, Today (4 hours ago)
Pharo image: PharoCore
Pharo core version: v
Virtual machine used: Squeak 4.2.2beta1U.app



Steps to reproduce:

1. Open several windows (whatever)
2. Close all of them
3. Press cmd + w (like trying to close another window).

You will get the attached error.

The problem is that the message closeTopWindow is being sent to
SystemWindow and this is not impelented. I look in older version and it is
like this:


SystemWindow class >> closeTopWindow
        "Try to close the top window.  It may of course decline"
        TopWindow ifNotNil:
                [TopWindow delete]


So, adding this method the error is dissappear, but:

1) It even works creating the message doing nothing
2) I think this is not the correct fix. The real fix should do some
ifEmpty:  don't send the message closeTopWindow or something like that.
3) I don't understand why the closing of OTHERS windows than the last one,
use another message then closeTopWindow



_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project