[squeak-dev] The Trunk: Collections-fn.869.mcz

Marcel Taeumel marcel.taeumel at hpi.de
Mon Jan 6 08:47:16 UTC 2020


+1 for this feature-freeze compatible feature. ;-P

Best,

Marcel
Am 04.01.2020 22:50:11 schrieb Chris Muller <asqueaker at gmail.com>:
+1   The purpose of feature freeze is to reduce risk, so if we feel comfortable enough about a change being functionally non-impactful, IMO, it seems fine to let it in the release.  This seems pretty cut 'n' dry safe to me, nice catch Fabio.

On Fri, Jan 3, 2020 at 5:43 PM Fabio Niephaus <lists at fniephaus.com [mailto:lists at fniephaus.com]> wrote:

Hi Levente,

On Fri, Jan 3, 2020 at 11:55 PM Levente Uzonyi <leves at caesar.elte.hu [mailto:leves at caesar.elte.hu]> wrote:

It's a nice improvement, but isn't it feature freeze with "only bugfixes,
but no new features"?


I was considering this one a performance bugfix, not a new feature. Not sure how we treated these simple cases in the past.

Fabio
 

Levente

On Fri, 3 Jan 2020, commits at source.squeak.org [mailto:commits at source.squeak.org] wrote:

> Fabio Niephaus uploaded a new version of Collections to project The Trunk:
> http://source.squeak.org/trunk/Collections-fn.869.mcz [http://source.squeak.org/trunk/Collections-fn.869.mcz]
>
> ==================== Summary ====================
>
> Name: Collections-fn.869
> Author: fn
> Time: 3 January 2020, 10:27:08.945712 am
> UUID: c29ef38b-f152-468e-bac9-74515c099ac4
> Ancestors: Collections-nice.868
>
> Add quick-check for identity to String>>sameAs: (similar to Character>>sameAs:).
>
> Example:
> [MenuIcons smallInspectItIconContents sameAs: MenuIcons smallInspectItIconContents] bench
>
> Before:
> 1,740,000 per second. 574 nanoseconds per run. 0 % GC time.
>
> After:
> 94,800,000 per second. 10.5 nanoseconds per run. 0 % GC time.
>
> =============== Diff against Collections-nice.868 ===============
>
> Item was changed:
>  ----- Method: String>>sameAs: (in category 'comparing') -----
>  sameAs: aString
>       "Answer whether the receiver sorts equal to aString. The
>       collation sequence is ascii with case differences ignored."
> +     self == aString ifTrue: [ ^true ].
>       ^(self compare: aString caseSensitive: false) = 2!


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20200106/23729a23/attachment.html>


More information about the Squeak-dev mailing list