The Inbox: Environments-fbs.5.mcz

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

The Inbox: Environments-fbs.5.mcz

commits-2
A new version of Environments was added to project The Inbox:
http://source.squeak.org/inbox/Environments-fbs.5.mcz

==================== Summary ====================

Name: Environments-fbs.5
Author: fbs
Time: 3 August 2012, 9:11:57.622 pm
UUID: 07a930ba-2277-4253-8810-25012ba8e67f
Ancestors: Environments-cmm.4

Colin Putney will point out that Environments should actually provide a higher-level API than Dictionary. This is a compatibility call.

=============== Diff against Environments-cmm.4 ===============

Item was added:
+ ----- Method: Environment>>removeKey:ifAbsent: (in category 'accessing') -----
+ removeKey: key ifAbsent: aBlock
+ self flag: #review.
+ ^ contents removeKey: key ifAbsent: aBlock!


Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Environments-fbs.5.mcz

Frank Shearar-3
On 3 August 2012 21:12,  <[hidden email]> wrote:

> A new version of Environments was added to project The Inbox:
> http://source.squeak.org/inbox/Environments-fbs.5.mcz
>
> ==================== Summary ====================
>
> Name: Environments-fbs.5
> Author: fbs
> Time: 3 August 2012, 9:11:57.622 pm
> UUID: 07a930ba-2277-4253-8810-25012ba8e67f
> Ancestors: Environments-cmm.4
>
> Colin Putney will point out that Environments should actually provide a higher-level API than Dictionary. This is a compatibility call.
>
> =============== Diff against Environments-cmm.4 ===============
>
> Item was added:
> + ----- Method: Environment>>removeKey:ifAbsent: (in category 'accessing') -----
> + removeKey: key ifAbsent: aBlock
> +       self flag: #review.
> +       ^ contents removeKey: key ifAbsent: aBlock!

In the interests of unblocking Dale, is someone other than I happy for
this change to hit Trunk?

frank

Reply | Threaded
Open this post in threaded view
|

Re: The Inbox: Environments-fbs.5.mcz

Hannes Hirzel
Yes, please add it to trunk. Adding #removeKey:ifAbsent# to
Environment is not a problem for me.

HJH

On 8/6/12, Frank Shearar <[hidden email]> wrote:

> On 3 August 2012 21:12,  <[hidden email]> wrote:
>> A new version of Environments was added to project The Inbox:
>> http://source.squeak.org/inbox/Environments-fbs.5.mcz
>>
>> ==================== Summary ====================
>>
>> Name: Environments-fbs.5
>> Author: fbs
>> Time: 3 August 2012, 9:11:57.622 pm
>> UUID: 07a930ba-2277-4253-8810-25012ba8e67f
>> Ancestors: Environments-cmm.4
>>
>> Colin Putney will point out that Environments should actually provide a
>> higher-level API than Dictionary. This is a compatibility call.
>>
>> =============== Diff against Environments-cmm.4 ===============
>>
>> Item was added:
>> + ----- Method: Environment>>removeKey:ifAbsent: (in category 'accessing')
>> -----
>> + removeKey: key ifAbsent: aBlock
>> +       self flag: #review.
>> +       ^ contents removeKey: key ifAbsent: aBlock!
>
> In the interests of unblocking Dale, is someone other than I happy for
> this change to hit Trunk?
>
> frank
>
>