Handle '/' in locale name in PAL#46601
Handle '/' in locale name in PAL#46601tarekgh merged 1 commit intodotnet:masterfrom am11:feature/globalization-native
Conversation
src/libraries/Native/Unix/System.Globalization.Native/pal_locale.c
Outdated
Show resolved
Hide resolved
|
@am11 could you please hold on this? I am looking if we can have a better fix here. |
|
@am11 I think we can go ahead with this fix here as it address the infinite looping and we can address the locale name validation in another PR. The reason here is we may decide later porting the infinite loop issue to 5.0. what you think? |
|
@tarekgh, I think it makes sense to keep the fix for infinite loop separate from exhaustive validation checks. Is it so that ICU is validating some characters but not all that forbidden by BCP47 for locale names? or are these validations also implemented in the runtime code? |
We manually handle the names containing |
|
@am11 thanks again for your help here. Are you interested to look at locale name validation when using ICU? |
|
@tarekgh, I can take a look at validation next. |
Fixes #46577