+1
Keep it consistent with the rest of the system.

On Mon, Aug 12, 2013 at 4:40 AM, Esteban Lorenzano <[email protected]>wrote:

> #withOccurrencesDo: matches better the rest of the pharo system
> (#withIndexDo:, etc) :)
>
> On Aug 12, 2013, at 9:16 AM, Stéphane Ducasse <[email protected]>
> wrote:
>
>
> On Aug 12, 2013, at 1:49 AM, Gabriel Cotelli <[email protected]> wrote:
>
> Why not #doWithOccurrences: ??
>
>
> ok it was a proposal for a new selector :)
>
>
> Eg:
> bag doWithOccurrences: [:element :count | "do something"]
>
> I dont'like the keysAndValuesDo: idea, is just an implementation detail.
>
>
> On Sun, Aug 11, 2013 at 4:52 PM, Stéphane Ducasse <
> [email protected]> wrote:
>
>> Hi guys
>>
>> I was surprised to see the API of bag for iteration.
>>
>> I propose to add
>>
>> keysAndValuesDo: aBlock
>>         "Iterate over the receiver and apply a two argument block on the
>> element and its occurrences."
>>
>>         contents associationsDo: [:assoc | aBlock value: assoc key value:
>> assoc value ]
>>
>> and
>>
>> elementsAndOccurrencesDo: aTwoArgBlock
>>         "Iterate over the receiver and apply a two argument block on the
>> element and its occurrences."
>>
>>         contents associationsDo: [:assoc | aTwoArgBlock value: assoc key
>> value: assoc value ]
>>
>> Once we agree I will add a bug entry.
>>
>> Stef
>>
>
>
>
>

Reply via email to