<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000;text-align: left" dir="ltr">
Hi Chris --<div><br></div><div>> <span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Text is concerned with presentation -- attributes, colors, fonts -- and not really the domain side,</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">> which is what #format: is dealing with.</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">String and Text are polymorphic in many cases. The use of #format: makes GUI scripting easier in</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">several cases. The reason for using {1} over {FOO} is unrelated to that. Therefore, it makes sense</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">to also add that feature to Text >> #format:.</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Let's open another thread for that:</span></div><div><a href="http://lists.squeakfoundation.org/pipermail/squeak-dev/2022-July/221586.html"><span style="font-size: 10pt;font-family: Arial, Helvetica, sans-serif">http://lists.squeakfoundation.org/pipermail/squeak-dev/2022-July/221586.html</span></a><br></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">> </span><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">See Collections-cmm.1018, which restores (mostly) the legacy performance when using</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">> numeric-only tokens.</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Nice! Thanks. :-)</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">> </span><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Speaking of which, WHAT in the heck happened to comment formatting in 6.0?</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">See other thread. If something got broken unintentionally, we can surely fix it:</span></div><div><a href="http://lists.squeakfoundation.org/pipermail/squeak-dev/2022-July/221585.html"><span style="font-size: 10pt;font-family: Arial, Helvetica, sans-serif">http://lists.squeakfoundation.org/pipermail/squeak-dev/2022-July/221585.html</span></a><br></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 13.07.2022 03:17:47 schrieb Chris Muller <asqueaker@gmail.com>:</p><div style="font-family:Arial,Helvetica,sans-serif">
<div dir="ltr"><div dir="ltr">Hi Marcel,</div><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex;border-left: 1px solid rgb(204,204,204);padding-left: 1ex;min-width: 500px"><div id="gmail-m_1161276196122988614__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: rgb(0,0,0);text-align: left" dir="ltr"><div>Please make sure to have Text >> #format: be on par with your changes.</div></div></blockquote><div><br></div><div>Hmm, I noticed that, but am hesitant to continue encouraging Text to take on that responsibility. Text is concerned with presentation -- attributes, colors, fonts -- and not really the domain side, which is what #format: is dealing with. I see the existing implementation is optimized by directly accessing its 'runs', which I would not do.</div><div> </div><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex;border-left: 1px solid rgb(204,204,204);padding-left: 1ex;min-width: 500px"><div id="gmail-m_1161276196122988614__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: rgb(0,0,0);text-align: left" dir="ltr"><div><span style="font-size: 10pt">Note that you changes put more pressure on the GC:</span><br></div><div><br></div><div><div><span style="font-size: 13.3333px">['Hello {1}!' format: { 'Squeak' }] bench</span></div><div><span style="font-size: 13.3333px"><br></span></div><div><span style="font-size: 13.3333px">BEFORE: '2,320,000 per second. 432 nanoseconds per run. 1.63967 % GC time.' </span></div><div><span style="font-size: 13.3333px">AFTER: '1,340,000 per second. 746 nanoseconds per run. 16.92 % GC time.' </span></div></div></div></blockquote><div><br></div><div>See Collections-cmm.1018, which restores (mostly) the legacy performance when using numeric-only tokens. The garbage you saw is necessary when using alphanumeric tokens. This optimization complexifies the code to the limit of what I believe is reasonable to achieve extra performance. I think people generally expect String manipulation to involve the need to produce some garbage.</div><div> </div><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex;border-left: 1px solid rgb(204,204,204);padding-left: 1ex;min-width: 500px"><div id="gmail-m_1161276196122988614__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: rgb(0,0,0);text-align: left" dir="ltr"><div><span style="font-size: 10pt">P.S.: </span><span style="font-size: 10pt">If you had not opted for re-layouting the entire method, I would be able to actually see what you changed there... ;-) ... pretty-print for obfuscation, I suppose? :-D</span><br></div></div></blockquote><div><br></div><div>For obfuscation? Give me a break, absolutely not. Huh? No, I've always been a supporter of preserving the formats of original works, especially when only superficial changes are being made, but this is a long method and the change is too complex for me to be expected not to utilize what I need to be productive, which includes the use of my pretty printer, yes.</div><div><br></div><div>Speaking of which, WHAT in the heck happened to comment formatting in 6.0? We've held off improving the pretty printer to emulate Rectangular Block for years out of respect for original authors chosen legacy format. But this new comment formatting blows that out of the water by changing peoples' **custom-formatted** comments to a horrendously narrow, super-tall format! Please revert ParseNode>>#printSingleComment:on:indent:.</div><div><br></div><div>Best,</div><div> Chris</div></div></div>
</div></blockquote></div>