dotnet / runtime

.NET is a cross-platform runtime for cloud, mobile, desktop, and IoT apps.
https://docs.microsoft.com/dotnet/core/
MIT License
15.44k stars 4.76k forks source link

[iOS/tvOS/MacCatalyst] String comparer for sorting numeric strings logically #109999

Open PranavSenthilnathan opened 5 days ago

PranavSenthilnathan commented 5 days ago

CompareOptions.NumericOrdering should be supported in Apple platforms. See #13979 for the approved API and discussion. PR #109861 adds the member CompareOptions.NumericOrdering but does not provide a conforming implementation for Apple platforms. The remaining work is to update the platform abstraction layer to support this.

PAL change would be to recognize the dotnet CompareOptions and convert it to the NSString.CompareOptions here

This is the numeric compare option documentation for reference: NSString.CompareOptions API

PR #109861 throws PlatformNotSupportedException when this option is used and skips tests that used this option, so when this is implemented, those guards can be removed.

dotnet-policy-service[bot] commented 5 days ago

Tagging subscribers to this area: @dotnet/area-system-globalization See info in area-owners.md if you want to be subscribed.

dotnet-policy-service[bot] commented 5 days ago

Tagging subscribers to 'os-tvos': @vitek-karas, @kotlarmilos, @ivanpovazan, @steveisok, @akoeplinger See info in area-owners.md if you want to be subscribed.

dotnet-policy-service[bot] commented 5 days ago

Tagging subscribers to 'os-ios': @vitek-karas, @kotlarmilos, @ivanpovazan, @steveisok, @akoeplinger See info in area-owners.md if you want to be subscribed.

dotnet-policy-service[bot] commented 5 days ago

Tagging subscribers to 'os-maccatalyst': @vitek-karas, @kotlarmilos, @ivanpovazan, @steveisok, @akoeplinger See info in area-owners.md if you want to be subscribed.