Issue 5797 in pharo: Adding announcements on open and delete for morphs

Status: Accepted
Owner: [hidden email]
Labels: Type-Enh Milestone-2.0

New issue 5797 by [hidden email]: Adding announcements on open  
and delete for morphs
http://code.google.com/p/pharo/issues/detail?id=5797

step by step :)


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

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 5797 by [hidden email]: Adding announcements  
on open and delete for morphs
http://code.google.com/p/pharo/issues/detail?id=5797

Slice in inbox!

Name:  
SLICE-Issue-5797-Adding-announcements-on-open-and-delete-for-morphs-GuillermoPolito.1
Author: GuillermoPolito
Time: 3 May 2012, 9:11:01.978 pm
UUID: 166fd37c-ab98-4764-81aa-16e90d94a754
Ancestors:
Dependencies: Polymorph-Widgets-GuillermoPolito.629,  
Morphic-GuillermoPolito.1140

issue  
http://code.google.com/p/pharo/issues/detail?id=5797&thanks=5797&ts=1336072208


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

Comment #2 on issue 5797 by [hidden email]: Adding announcements on  
open and delete for morphs
http://code.google.com/p/pharo/issues/detail?id=5797

How much slower does this make the system?


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

Comment #3 on issue 5797 by [hidden email]: Adding announcements  
on open and delete for morphs
http://code.google.com/p/pharo/issues/detail?id=5797

I'm not an expert of benchmarcks but with the following code opening 10  
nautilus (which is preety heavy)

[
        1to: 10 do: [ :i |
                | b |
                b := Smalltalk tools browser open.
                b ui window delete.
                ]
] timeToRun.

I got

"6 runs without announcements"
7540 7372 7370 7104 7590 7820.

"6 runs with announcements"
7556 7102 7532 7614 7739 7730


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

Comment #4 on issue 5797 by [hidden email]: Adding announcements on  
open and delete for morphs
http://code.google.com/p/pharo/issues/detail?id=5797

Marcus, why care? How many morphs we creating/deleting when running UI?  
Thousands? Millions? We're talking about 10 to 100 morphs.. and time to  
build UIs with it + announcements will be hardly noticeable by users.


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

Updates:
        Cc: [hidden email]

Comment #5 on issue 5797 by [hidden email]: Adding announcements  
on open and delete for morphs
http://code.google.com/p/pharo/issues/detail?id=5797

Do we integrate it? :3  So we can do a second iteration...


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

Updates:
        Status: FixToInclude

Comment #6 on issue 5797 by [hidden email]: Adding announcements on  
open and delete for morphs
http://code.google.com/p/pharo/issues/detail?id=5797

(No comment was entered for this change.)


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

Updates:
        Status: Integrated

Comment #7 on issue 5797 by [hidden email]: Adding announcements on  
open and delete for morphs
http://code.google.com/p/pharo/issues/detail?id=5797

in 20 089


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