<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000">
                                        
                                        
                                            
                                        
                                        
                                        Hi Chris,<div><br></div><div>-1 for moving #join etc. up to Collection. You seem to favor some performance improvement over code readability. Making the order of the collection explicit before joining it feels more robust. #joinSeparatedBy: on non-sequencable collections has to much potential for bugs and unreadable code. See <a href="http://forum.world.st/The-Inbox-Collections-ct-827-mcz-tp5099876p5099964.html" style="font-size: 10pt;line-height: 1.5">http://forum.world.st/The-Inbox-Collections-ct-827-mcz-tp5099876p5099964.html</a> -- Having a non-seq. collection to join should not be a common case.</div><div><br></div><div>-1 for mixing up separate concerns in a single commit. Please, try separating those things.</div><div><br></div><div>Best,</div><div>Marcel</div><div class="mb_sig"></div>
                                        
                                        <blockquote class="history_container" type="cite" style="border-left-style: solid;border-width: 1px;margin-top: 20px;margin-left: 0px;padding-left: 10px;min-width: 500px">
                        <p style="color: #AAAAAA; margin-top: 10px;">Am 05.06.2019 00:14:53 schrieb Chris Muller <asqueaker@gmail.com>:</p><div style="font-family:Arial,Helvetica,sans-serif">Hi Nicolas,<br><br>> I don't much like WriteStream on: String empty.<br>> If String empty is implemented as answering a literal, and WritStream implemented to become the grown contents as it originally did in st80 and could also do in Squeak now that we have a fast become, we take the risk to modify the literal...<br><br>We both know that's not going to happen.  #empty has been a class-side<br>constructor/accessor on several classes for a long time, with > 100<br>senders in trunk alone.  It's an integrated part of the<br>class-library's API which improves performance and readability.  Any<br>attempt to switch back to a become implementation for growing internal<br>collections would spoil that API everywhere.<br><br>> String new better express our intention, we want a new substring.<br><br>But sometimes we trade intention-revealingness for performance.<br>That's what this is.<br><br>I would offer the compromise of WriteStream on: '', but with #empty<br>being an integrated part of the API, it's the intent as everywhere<br>else it's used.<br><br>> Also, contents does not necessarily answer a copy,<br><br>But it does.  WriteStream>>#contents answers a #copyFrom:to: and even<br>the comment says "Answer with a copy ..."<br><br> - Chris<br><br></div></blockquote></div>