I am ok to this change. > -----Original Message----- > From: Gao, Zhichao <[email protected]> > Sent: Thursday, October 24, 2019 9:25 AM > To: Marvin Häuser <[email protected]>; [email protected] > Cc: [email protected]; Ni, Ray <[email protected]> > Subject: RE: [PATCH] UefiShellCommandLib: Default to first found UC for > unsupported PlatformLang > > Hi Ray, > > This patch would set the default language of shell to the first found language > instead of ASSERT when the matched language is not found. What do you > think of this change? I don't know the reason of assert. If it is required to > ASSERT to show the user the shell language should be matched with the > platform language. Then the patch is inappropriate. If not, the patch is fine. > > Thanks, > Zhichao > > > -----Original Message----- > > From: Marvin Häuser [mailto:[email protected]] > > Sent: Sunday, October 20, 2019 8:09 PM > > To: [email protected] > > Cc: [email protected]; Ni, Ray <[email protected]>; Gao, Zhichao > > <[email protected]> > > Subject: [PATCH] UefiShellCommandLib: Default to first found UC for > > unsupported PlatformLang > > > > From: Marvin Haeuser <[email protected]> > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2300 > > > > On some firmwares PlatformLang is set to the local language (e.g. > > ru-RU), however there is no Unicode Collation protocol instance that > supports it. > > As for missing PlatformLang, fall back to the first found instance. > > > > Cc: Ray Ni <[email protected]> > > Cc: Zhichao Gao <[email protected]> > > Signed-off-by: Marvin Haeuser <[email protected]> > > --- > > ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c | 6 > +++++- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git > > a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c > > b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c > > index 4c48b65fbc1d..345808a1eac6 100644 > > --- a/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c > > +++ b/ShellPkg/Library/UefiShellCommandLib/UefiShellCommandLib.c > > @@ -107,9 +107,13 @@ CommandInit( > > // // Without clue provided use the first Unicode Collation2 > protocol.+ > > // This may happen when PlatformLang is NULL or when no installed > > Unicode+ // Collation2 protocol instance supports PlatformLang. > > //- > if > > (PlatformLang == NULL) {+ if (gUnicodeCollation == NULL) > > { gUnicodeCollation = Uc;+ }+ if (PlatformLang == NULL) > > { break; } -- > > 2.23.0.windows.1
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#49949): https://edk2.groups.io/g/devel/message/49949 Mute This Topic: https://groups.io/mt/35954837/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
