Issue 4978 in pharo: DNU: RGMethodDefinition>>first

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

Issue 4978 in pharo: DNU: RGMethodDefinition>>first

pharo
Status: Accepted
Owner: [hidden email]

New issue 4978 by [hidden email]: DNU: RGMethodDefinition>>first
http://code.google.com/p/pharo/issues/detail?id=4978

When removing a method in latest 1.4.

The problem is happening on:

FlatMessageListBrowser>>methodRemoved: ann
        | ref |
        ref := self getList detect: [:any | (any first realClass = ann item  
methodClass) and: [ any last = ann item selector ]] ifNone: [ nil ].
        ref ifNotNil: [ | newRef |
                        self deleteFromMessageList: (ref first methodNamed: ref last).
                        self updateView]

It seems now the list contains RGMethodDefinition instances instead of the  
previous arrays :/.
Something similar happens in the implementors browser


_______________________________________________
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 4978 in pharo: DNU: RGMethodDefinition>>first

pharo
Updates:
        Status: FixReviewNeeded
        Cc: [hidden email]

Comment #1 on issue 4978 by [hidden email]: DNU:  
RGMethodDefinition>>first
http://code.google.com/p/pharo/issues/detail?id=4978

Yes we removed this ugly first second idea and this is probably a left over.
We will fix it.


_______________________________________________
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 4978 in pharo: DNU: RGMethodDefinition>>first

pharo

Comment #2 on issue 4978 by [hidden email]: DNU:  
RGMethodDefinition>>first
http://code.google.com/p/pharo/issues/detail?id=4978

Patch :)

Thanks Guillermo for reporting :)

Attachments:
        RemoveSomeFirsts.1.cs  640 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 4978 in pharo: DNU: RGMethodDefinition>>first

pharo

Comment #3 on issue 4978 by [hidden email]: DNU:  
RGMethodDefinition>>first
http://code.google.com/p/pharo/issues/detail?id=4978

Thanks. I just finished to reorganize my hifi system and connected it with  
my ps2 :)
I will integrate it now.



_______________________________________________
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 4978 in pharo: DNU: RGMethodDefinition>>first

pharo
Updates:
        Status: Closed

Comment #4 on issue 4978 by [hidden email]: DNU:  
RGMethodDefinition>>first
http://code.google.com/p/pharo/issues/detail?id=4978

in 14220


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