core icon indicating copy to clipboard operation
core copied to clipboard

[Consistency Review] There's no Array::stable_sort

Open hackwaly opened this issue 6 months ago • 1 comments

Current behaviors

There's FixedArray::stable_sort, but not Array::stable_sort

FixedArray::stable_sort_by and Array::stable_sort_by is also missing.

Expected behaviors

There should be Array::stable_sort

Justification

hackwaly avatar Aug 25 '25 15:08 hackwaly

It seems unstable version not always significantly faster than stable version.

I think we should expose only one sort func sticky on stable version instead of two sort and stable_sort.

If unstable version is much faster, we could provide another fast_sort func.

hackwaly avatar Aug 25 '25 15:08 hackwaly