Process>>#'environmentAt:put:'

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

Process>>#'environmentAt:put:'

jgfoster
The typical Dictionary protocol is to return the added value rather  
than self. I suggest that Process>>#'environmentAt:put:' follow the  
same convention.

Dictionary new at: #'James' put: 1. "returns 1"
Processor activeProcess environmentAt: #'James' put: 1. "returns  
aProcess"



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

Re: Process>>#'environmentAt:put:'

Stéphane Ducasse
Good point
        http://code.google.com/p/pharo/issues/list?thanks=1117

On Aug 30, 2009, at 12:43 PM, James Foster wrote:

> The typical Dictionary protocol is to return the added value rather
> than self. I suggest that Process>>#'environmentAt:put:' follow the
> same convention.
>
> Dictionary new at: #'James' put: 1. "returns 1"
> Processor activeProcess environmentAt: #'James' put: 1. "returns
> aProcess"
>
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


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

Re: Process>>#'environmentAt:put:'

Giovanni Corriga
SLICE-ProcessEnvironmentFixes-GiovanniCorriga.1 fixes that and adds a
few tests.

        Giovanni

Stéphane Ducasse ha scritto:

> Good point
> http://code.google.com/p/pharo/issues/list?thanks=1117
>
> On Aug 30, 2009, at 12:43 PM, James Foster wrote:
>
>> The typical Dictionary protocol is to return the added value rather
>> than self. I suggest that Process>>#'environmentAt:put:' follow the
>> same convention.
>>
>> Dictionary new at: #'James' put: 1. "returns 1"
>> Processor activeProcess environmentAt: #'James' put: 1. "returns
>> aProcess"
>>
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


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

Re: Process>>#'environmentAt:put:'

Stéphane Ducasse
Great!

Stef

On Aug 30, 2009, at 5:52 PM, Giovanni Corriga wrote:

> SLICE-ProcessEnvironmentFixes-GiovanniCorriga.1 fixes that and adds a
> few tests.
>
> Giovanni
>
> Stéphane Ducasse ha scritto:
>> Good point
>> http://code.google.com/p/pharo/issues/list?thanks=1117
>>
>> On Aug 30, 2009, at 12:43 PM, James Foster wrote:
>>
>>> The typical Dictionary protocol is to return the added value rather
>>> than self. I suggest that Process>>#'environmentAt:put:' follow the
>>> same convention.
>>>
>>> Dictionary new at: #'James' put: 1. "returns 1"
>>> Processor activeProcess environmentAt: #'James' put: 1. "returns
>>> aProcess"
>>>
>>>
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


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