Issue 3811 in pharo: Extending MessageSend protocol with cull: and value: protocol

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

Issue 3811 in pharo: Extending MessageSend protocol with cull: and value: protocol

pharo
Status: Accepted
Owner: [hidden email]

New issue 3811 by [hidden email]: Extending MessageSend protocol with  
cull: and value: protocol
http://code.google.com/p/pharo/issues/detail?id=3811

We need it for Announcements

Attachments:
        cull-cull.1.cs  1.2 KB


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3811 in pharo: Extending MessageSend protocol with cull: and value: protocol

pharo
Updates:
        Status: Fixed
        Cc: stephane.ducasse tudor.girba
        Labels: Milestone-1.3

Comment #1 on issue 3811 by [hidden email]: Extending MessageSend  
protocol with cull: and value: protocol
http://code.google.com/p/pharo/issues/detail?id=3811

in order to treat MessageSends and blocks in same way
so, we need support of these messages in MessageSend.

And since MessageSend already supports valueWithArguments:,
we also extended it with faster #value: #value:value: and  
#value:value:value:


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3811 in pharo: Extending MessageSend protocol with cull: and value: protocol

pharo

Comment #2 on issue 3811 by [hidden email]: Extending MessageSend  
protocol with cull: and value: protocol
http://code.google.com/p/pharo/issues/detail?id=3811

You will integrate it because you should join. We should not be the  
bottleneck.
I will review the code and we will integrate it together monday.


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3811 in pharo: Extending MessageSend protocol with cull: and value: protocol

pharo
In reply to this post by pharo
Issue 3811: Extending MessageSend protocol with cull: and value: protocol
http://code.google.com/p/pharo/issues/detail?id=3811

This issue is now blocking issue 3814.
See http://code.google.com/p/pharo/issues/detail?id=3814

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

Reply | Threaded
Open this post in threaded view
|

Re: Issue 3811 in pharo: Extending MessageSend protocol with cull: and value: protocol

pharo
In reply to this post by pharo

Comment #3 on issue 3811 by [hidden email]: Extending MessageSend  
protocol with cull: and value: protocol
http://code.google.com/p/pharo/issues/detail?id=3811

SLICE-Issue-3811-cull-protocol-for-MessageSends-HenrikSperreJohansen.1 in  
Inbox.

In addition to what is in the .cs above, it also includes protocols for  
WeakMessageSend, which would otherwise be found in  
Announcements-phase2.3.cs in Issue 3814


Reply | Threaded
Open this post in threaded view
|

Re: Issue 3811 in pharo: Extending MessageSend protocol with cull: and value: protocol

pharo
Updates:
        Status: Closed

Comment #4 on issue 3811 by [hidden email]: Extending MessageSend  
protocol with cull: and value: protocol
http://code.google.com/p/pharo/issues/detail?id=3811

in 13088