<div dir="ltr"><div>Hi Fabio, Levente,</div><div>I'd like to proceed with restoring input interpreter health along with promotion of new VM.</div><div>IMO pure optimization can go in trunk if not too complex, nor raise too many questions.</div><div>RunArray optimization can wait for example.</div><div><br></div><div>I'd like to underline the tremendous work performed by Marcel so far.<br></div><div>It's possible that the release was a bit frozen because he took a few days off.</div><div>These were well-earned!<br></div><div><br></div><div><br></div><div><br></div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le sam. 4 janv. 2020 à 04:32, Levente Uzonyi <<a href="mailto:leves@caesar.elte.hu" target="_blank">leves@caesar.elte.hu</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Fabio,<br>
<br>
I don't remember having so many new entries in the Inbox during previous <br>
releases. Also, I feel like this release is taking a bit too long to be <br>
ready.<br>
I asked you about your commit because there are a few other "performance <br>
bugfixes" waiting in the Inbox, but I'm not sure if they should be pushed <br>
now or not.<br>
Also, that indentity check you just added to #sameAs: could be added to <br>
other related comparison methods too (#compare:caseSensitive:, <br>
#compare:with:collated:, even to the primitive #primitiveCompareString).<br>
<br>
<br>
Levente<br>
<br>
On Sat, 4 Jan 2020, Fabio Niephaus wrote:<br>
<br>
> Hi Levente,<br>
> <br>
> On Fri, Jan 3, 2020 at 11:55 PM Levente Uzonyi <<a href="mailto:leves@caesar.elte.hu" target="_blank">leves@caesar.elte.hu</a>> wrote:<br>
>       It's a nice improvement, but isn't it feature freeze with "only bugfixes,<br>
>       but no new features"?<br>
> <br>
> <br>
> I was considering this one a performance bugfix, not a new feature. Not sure how we treated these simple cases in the past.<br>
> <br>
> Fabio<br>
>  <br>
><br>
>       Levente<br>
><br>
>       On Fri, 3 Jan 2020, <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> wrote:<br>
><br>
>       > Fabio Niephaus uploaded a new version of Collections to project The Trunk:<br>
>       > <a href="http://source.squeak.org/trunk/Collections-fn.869.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/trunk/Collections-fn.869.mcz</a><br>
>       ><br>
>       > ==================== Summary ====================<br>
>       ><br>
>       > Name: Collections-fn.869<br>
>       > Author: fn<br>
>       > Time: 3 January 2020, 10:27:08.945712 am<br>
>       > UUID: c29ef38b-f152-468e-bac9-74515c099ac4<br>
>       > Ancestors: Collections-nice.868<br>
>       ><br>
>       > Add quick-check for identity to String>>sameAs: (similar to Character>>sameAs:).<br>
>       ><br>
>       > Example:<br>
>       > [MenuIcons smallInspectItIconContents sameAs: MenuIcons smallInspectItIconContents] bench<br>
>       ><br>
>       > Before:<br>
>       > 1,740,000 per second. 574 nanoseconds per run. 0 % GC time.<br>
>       ><br>
>       > After:<br>
>       > 94,800,000 per second. 10.5 nanoseconds per run. 0 % GC time.<br>
>       ><br>
>       > =============== Diff against Collections-nice.868 ===============<br>
>       ><br>
>       > Item was changed:<br>
>       >  ----- Method: String>>sameAs: (in category 'comparing') -----<br>
>       >  sameAs: aString<br>
>       >       "Answer whether the receiver sorts equal to aString. The<br>
>       >       collation sequence is ascii with case differences ignored."<br>
>       > +     self == aString ifTrue: [ ^true ].<br>
>       >       ^(self compare: aString caseSensitive: false) = 2!<br>
> <br>
> <br>
><br>
</blockquote></div>