Issue 7420 in pharo: Random behavior in ListComposableModel>>whenListChanged:

Status: New
Owner: ----
Labels: Type-Bug Target-Spec Milestone-2.0 Difficulty-Easy

New issue 7420 by [hidden email]: Random behavior in  
ListComposableModel>>whenListChanged:
http://code.google.com/p/pharo/issues/detail?id=7420

Pharo image: 20463

To reproduce: run

50 timesRepeat: [|list|
list := ListComposableModel new.
list items: #(1 2).
list whenListChanged: [:aList|
Transcript crShow: aList = list listItems].
list listItems. "Call listItems to set the cache"
list items: #(1 2 3) ].

You should allways get true in the Transcript, instead you get a mix of  
true and false.


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

Comment #1 on issue 7420 by [hidden email]: Random behavior in  
ListComposableModel>>whenListChanged:
http://code.google.com/p/pharo/issues/detail?id=7420

A test case showing the core effect.

| announcer ok |
announcer := Announcer new.
20
        timesRepeat: [
                ok := true.
                announcer
                        on: Announcement
                        do: [ ok := false ].
                announcer
                        on: Announcement
                        do: [ ok := true ].
                announcer announce: Announcement new.
                self assert: ok.
                announcer unsubscribe: self ]


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

Comment #2 on issue 7420 by [hidden email]: Random behavior in  
ListComposableModel>>whenListChanged:
http://code.google.com/p/pharo/issues/detail?id=7420

Slice in inbox: Change in ListComposableModel, with test cases.


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

Updates:
        Status: HumanReviewNeeded
        Cc: [hidden email]

Comment #3 on issue 7420 by [hidden email]: Random behavior in  
ListComposableModel>>whenListChanged:
http://code.google.com/p/pharo/issues/detail?id=7420

It makes the example above pass. Ben, what do you think?


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

Updates:
        Status: FixToInclude

Comment #4 on issue 7420 by [hidden email]: Random behavior in  
ListComposableModel>>whenListChanged:
http://code.google.com/p/pharo/issues/detail?id=7420

I valid this fix :)


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

Updates:
        Status: Integrated

Comment #5 on issue 7420 by [hidden email]: Random behavior in  
ListComposableModel>>whenListChanged:
http://code.google.com/p/pharo/issues/detail?id=7420

in 2.0 518


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