Issue 4989 in pharo: SubscriptionRegistry>>deliver: anAnnouncement

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

Issue 4989 in pharo: SubscriptionRegistry>>deliver: anAnnouncement

pharo
Status: FixReviewNeeded
Owner: ----
Labels: Milestone-1.4 Type-Feature

New issue 4989 by [hidden email]: SubscriptionRegistry>>deliver:  
anAnnouncement
http://code.google.com/p/pharo/issues/detail?id=4989

if there are no subscriptions, then simply return before protecting and  
delivering the announcement.

deliver: anAnnouncement
        |  subsCopy |
       
        "using a copy, so subscribers can unsubscribe from announcer "
        "Use OrderedCollection , so unwind protection is easier"
--> subscriptions isEmpty ifTrue:[ ^ self ].
        self protected: [ subsCopy := subscriptions as: OrderedCollection ].
        self deliver: anAnnouncement to: subsCopy! !


Attachments:
        SubscriptionRegistry-deliver.st  506 bytes


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

Re: Issue 4989 in pharo: SubscriptionRegistry>>deliver: anAnnouncement

pharo
Updates:
        Status: FixToInclude

Comment #1 on issue 4989 by [hidden email]:  
SubscriptionRegistry>>deliver: anAnnouncement
http://code.google.com/p/pharo/issues/detail?id=4989

(No comment was entered for this change.)


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

Re: Issue 4989 in pharo: SubscriptionRegistry>>deliver: anAnnouncement

pharo
Updates:
        Status: Integrated

Comment #2 on issue 4989 by [hidden email]:  
SubscriptionRegistry>>deliver: anAnnouncement
http://code.google.com/p/pharo/issues/detail?id=4989

in 14225


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