On 07/04/2011 12:05 PM, Gwenael Casaccio wrote:
So here is the patch.

This is not a patch, it is 10 patches including multiple reverts of pieces that you have already submitted.

You should have grouped the patches in two, one for #beConsistent and one squashing everything else.

I've changed OrederedCollection to use
#replaceFrom:to:with:startingAt, SortedCollection uses the Smalltalk
implementation but for growing I call the primitive. And OrderedSet uses
the St implementation.

Why? Is it just because the ordered_set_class is not available? I believe this rather shows that class checks are too specific.

I pulled some changes in stable-3.2 and others in master only. Please test master and see how performance compares with your code.

Paolo

_______________________________________________
help-smalltalk mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/help-smalltalk

Reply via email to