Added POSIX modifiers support for BCP47 conversion - Githubissues

1Password / sys-locale

A small and lightweight Rust library to obtain the active locale on the system.

Apache License 2.0

75 stars 15 forks source link

Added POSIX modifiers support for BCP47 conversion #33

Open pasabanov opened 15 hours ago

pasabanov commented 15 hours ago

Partially resolves #32.

Renamed function parse_locale_code to posix_to_bcp47.
Changed return type from Option<String> to String.
Changed locale cropping to trim at the first occurrence of either dot (.) or at sign (@), removing any modifiers.
Added tests for modifier cropping.
Added doc comment for posix_to_bcp47.

Possible further optimizations:

Consider changing the code parameter type from &str to String (making the function posix_to_bcp47 consuming).