<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">2014-05-26 21:00 GMT+02:00 Bert Freudenberg <span dir="ltr"><<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class=""><br>
On 2014-05-26, at 18:40, Levente Uzonyi <<a href="mailto:leves@elte.hu">leves@elte.hu</a>> wrote:<br>
<br>
> On Mon, 26 May 2014, Marcel Taeumel wrote:<br>
><br>
>> Should this work?<br>
>><br>
>> OrderedCollection streamContents: [:result |<br>
>> result nextPutAll: #(a b c) asOrderedCollection].<br>
>><br>
>> Well, this works:<br>
>><br>
>> OrderedCollection streamContents: [:result |<br>
>> result nextPutAll: #(a b c)].<br>
>><br>
>> And this works, too:<br>
>><br>
>> OrderedCollection streamContents: [:result |<br>
>> result nextPut: #foo; nextPutAll: #(a b c) asOrderedCollection].<br>
><br>
> Even if it would work, it's a bad pattern, don't use it. OrderedCollection implements its own streaming API (#add:, #addAll:), which is a lot more efficient. I would use the same API as in streams in my own Smalltalk, but that's another story.<br>
><br>
> Also, converting Arrays to OrderedCollections just to throw them away in the next step is just as bad as creating a stream over an OrderedCollection. Don't do it.<br>
><br>
><br>
> Levente<br>
<br>
</div>Still, it should work, don't you think?<br>
<span class="HOEnZb"><font color="#888888"><br>
- Bert -<br>
<br></font></span></blockquote><div><br></div><div>It would work if only the implementation of Stream was less bloated.<br></div></div>With current state of code, making it work might have a hard price for low reward IMO.<br>
</div><div class="gmail_extra">But I didn't look at exact details of the failure, maybe adding one more patch will do it... (until the next bug)<br><br><br></div></div>