Issue 5485 in pharo: Break dependency in SmalltalkImage>>snapshot:andQuit:

Status: Accepted
Owner: [hidden email]
Labels: Milestone-1.4

New issue 5485 by [hidden email]: Break dependency in  
SmalltalkImage>>snapshot:andQuit:
http://code.google.com/p/pharo/issues/detail?id=5485

we should get rid of:

Cursor write show.
Cursor normal show.

there, because it is ui-related.. and not directly related to the purpose  
of method itself.

Normally, there should be some outer code doing things like:

Cursor write showWhile: [
   Smalltalk snapshot and quit
]

which is easy to put into methods which reacting on menu 'save as/save and  
quit' etc.. instead of polluting the snapshot code with ui elements.


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

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 5485 by [hidden email]: Break dependency in  
SmalltalkImage>>snapshot:andQuit:
http://code.google.com/p/pharo/issues/detail?id=5485


I put the Cursor dependencies on the WorldState class invoking the snapshot.

Attachments:
        snapshot-dependencies.1.cs  4.6 KB


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

Updates:
        Status: FixToInclude

Comment #2 on issue 5485 by [hidden email]: Break dependency in  
SmalltalkImage>>snapshot:andQuit:
http://code.google.com/p/pharo/issues/detail?id=5485

that's much better :)


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

Updates:
        Status: Integrated
        Labels: Type-Cleanup

Comment #3 on issue 5485 by [hidden email]: Break dependency in  
SmalltalkImage>>snapshot:andQuit:
http://code.google.com/p/pharo/issues/detail?id=5485

in 14395


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