Reverted via Collections-ct.982.<br>
<br>
Best,<br>
Christoph<br>
<br>
<font color="#808080">---<br>
</font><font color="#808080"><i>Sent from </i></font><font color="#808080"><i><a href="https://github.com/hpi-swa-lab/squeak-inbox-talk"><u><font color="#808080">Squeak Inbox Talk</font></u></a></i></font><br>
<br>
On 2022-01-28T08:41:10+01:00, marcel.taeumel@hpi.de wrote:<br>
<br>
> Hi Christoph --<br>
> <br>
> You can do in your personal image want you desire. ;-) Even add such extra protocols to help you in your personal scripting tasks.<br>
> <br>
> For Trunk, however, you must restrain yourself a little bit more. There is a fine line between a so-called "logical implication" and what is actually required ... and what can be maintained.<br>
> <br>
> >  I don't think that these few convenience selectors in their own protocol actually hurt, do they?<br>
> <br>
> The fact that I raised this concern and explained it to you indicates, that it does actually hurt some of us. I would advise that you remove this again and explore a little bit more what level of convenience is actually needed.<br>
> <br>
> So, please remove this for now.<br>
> <br>
> Best,<br>
> Marcel<br>
> Am 27.01.2022 17:37:40 schrieb Thiede, Christoph <christoph.thiede at student.hpi.uni-potsdam.de>:<br>
> Hi Marcel,<br>
> <br>
> I understand that we are unlikely to reach full polymorph as long as every code needs to be duplicated. Traits for all the conveniences methods *might* be an option, but I also understand that we seem not to want to make use of Traits here. :-)<br>
> <br>
> Anyway, I use #collect:thenDo: as often as #select:thenCollect: and this inconsistency/ununderstood messages were striking me repeatedly. I don't think that these few convenience selectors in their own protocol actually hurt, do they? :-) Also, converting to a collection is not always an option when performance matters.<br>
> <br>
> Best,<br>
> Christoph<br>
> Von: Squeak-dev <squeak-dev-bounces at lists.squeakfoundation.org> im Auftrag von Taeumel, Marcel<br>
> Gesendet: Donnerstag, 27. Januar 2022 11:13:17<br>
> An: squeak-dev<br>
> Betreff: Re: [squeak-dev] The Trunk: Collections-ct.980.mcz<br>
>  <br>
> Hi Christoph --<br>
> <br>
> I don't think that we should port all convenience methods from Collection to here. Just the core. At least, that was my thinking, when I added the "collections - enumerating" back then. I deliberately chose not to add such things to not bloat the Stream interface any further.<br>
> <br>
> Best,<br>
> Marcel<br>
> Am 26.01.2022 21:46:25 schrieb commits at source.squeak.org <commits at source.squeak.org>:<br>
> Christoph Thiede uploaded a new version of Collections to project The Trunk:<br>
> http://source.squeak.org/trunk/Collections-ct.980.mcz<br>
> <br>
> ==================== Summary ====================<br>
> <br>
> Name: Collections-ct.980<br>
> Author: ct<br>
> Time: 26 January 2022, 9:46:11.770407 pm<br>
> UUID: 465bffe2-5612-4c40-974b-33bd5ec2dbbd<br>
> Ancestors: Collections-mt.978<br>
> <br>
> Improves polymorphy between Collection and Stream by adding missing convenience selectors on the latter. We already have #select:thenCollect: there, so this is just a logical complement.<br>
> <br>
> =============== Diff against Collections-mt.978 ===============<br>
> <br>
> Item was added:<br>
> + ----- Method: Stream>>collect:thenDo: (in category 'collections - enumerating') -----<br>
> + collect: collectBlock thenDo: doBlock<br>
> +<br>
> + ^ (self collect: collectBlock) do: doBlock!<br>
> <br>
> Item was added:<br>
> + ----- Method: Stream>>collect:thenSelect: (in category 'collections - enumerating') -----<br>
> + collect: collectBlock thenSelect: selectBlock<br>
> +<br>
> + ^ (self collect: collectBlock) select: selectBlock!<br>
> <br>
> Item was added:<br>
> + ----- Method: Stream>>reject:thenDo: (in category 'collections - enumerating') -----<br>
> + reject: rejectBlock thenDo: doBlock<br>
> +<br>
> + ^ (self reject: rejectBlock) do: doBlock!<br>
> <br>
> Item was added:<br>
> + ----- Method: Stream>>select:thenDo: (in category 'collections - enumerating') -----<br>
> + select: selectBlock thenDo: doBlock<br>
> +<br>
> + ^ (self select: selectBlock) do: doBlock!<br>
> <br>
> <br>
> -------------- next part --------------<br>
> An HTML attachment was scrubbed...<br>
> URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220128/21520965/attachment.html><br>
> <br>
>