[Cuis-dev] Bag>>#sum:ifEmpty:

[Hernán Wilkinson]

yeap, the current implementation is not correct.
Juan, attached is a change set that fixes it and another with the related
tests.

Cheers!
Hernan.

On Tue, Nov 1, 2022 at 3:39 AM Luciano Notarfrancesco <luchiano at gmail.com>
wrote:

> I’m afk right now so I cannot check, but it sounds like I made a mistake.
> Of course it should be the value of aBlock at each element times the number
> of occurrences.
>
> On Tue, 1 Nov 2022 at 07:33 Hernán Wilkinson <hernan.wilkinson at 10pines.com>
> wrote:
>
>> Hi,
>>  the implementation of Bag>>#sum: aBlock ifEmpty: emptyBlock does not use
>> the parameter aBlock at all and assumes that each of the elements answers
>> the message *
>>  @Luciano Notarfrancesco <luchiano at gmail.com> the implementation is
>> yours and it is very new? Is there a reason you did that way?
>>
>> Thanks
>> Hernan
>>
>>
>> --
>>
>> *Hernán WilkinsonAgile Software Development, Teaching & Coaching*
>> *Phone: +54-011*-4893-2057
>> *Twitter: @HernanWilkinson*
>> *site: http://www.10Pines.com <http://www.10pines.com/>*
>> Address: Alem 896, Floor 6, Buenos Aires, Argentina
>>
>

-- 

*Hernán WilkinsonAgile Software Development, Teaching & Coaching*
*Phone: +54-011*-4893-2057
*Twitter: @HernanWilkinson*
*site: http://www.10Pines.com <http://www.10pines.com/>*
Address: Alem 896, Floor 6, Buenos Aires, Argentina
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20221101/65d9674c/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 5547-BagSumIfEmpty-HernanWilkinson-2022Nov01-23h18m-HAW.001.cs.st
Type: application/octet-stream
Size: 176 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20221101/65d9674c/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: UnsavedChangesTo-BaseImageTests-HAW.001.cs.st
Type: application/octet-stream
Size: 928 bytes
Desc: not available
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20221101/65d9674c/attachment-0001.obj>