[AMD Official Use Only - General] > -----Original Message----- > From: Mike Maslenkin <mike.maslen...@gmail.com> > Sent: Wednesday, June 21, 2023 5:32 AM > To: devel@edk2.groups.io > Cc: Chang, Abner <abner.ch...@amd.com>; nick...@nvidia.com; > ig...@ami.com; Mike Maslenkin <mike.maslen...@gmail.com> > Subject: [PATCH] [edk2-redfish-client] Remove reference to BaseSortLib library > > Caution: This message originated from an External Source. Use proper caution > when opening attachments, clicking links, or responding. > > > There are two problems with BaseSortLib reference: > 1 BaseSortLib is the library instance name not the class name. > 2 UefiSortLib instance from MdeModulePkg is also referenced below. > > Let's use only one reference to SortLib. > Even though Redfish Client uses only quick sort for char* elements, > UefiSortLib is preferred as it allows to sort DevicePath instances and > wide strings.
Sure, thanks for the input. Reviewed-by: Abner Chang <abner.ch...@amd.com> Abner > > Signed-off-by: Mike Maslenkin <mike.maslen...@gmail.com> > Cc: Abner Chang <abner.ch...@amd.com> > Cc: Nickle Wang <nick...@nvidia.com> > Cc: Igor Kulchytskyy <ig...@ami.com> > --- > RedfishClientPkg/RedfishClientPkg.dsc | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/RedfishClientPkg/RedfishClientPkg.dsc > b/RedfishClientPkg/RedfishClientPkg.dsc > index edc387ac72dc..e16c91b83196 100644 > --- a/RedfishClientPkg/RedfishClientPkg.dsc > +++ b/RedfishClientPkg/RedfishClientPkg.dsc > @@ -36,7 +36,6 @@ > JsonLib|RedfishPkg/Library/JsonLib/JsonLib.inf > > Ucs2Utf8Lib|RedfishPkg/Library/BaseUcs2Utf8Lib/BaseUcs2Utf8Lib.inf > > RedfishCrtLib|RedfishPkg/PrivateLibrary/RedfishCrtLib/RedfishCrtLib.inf > > - BaseSortLib|MdeModulePkg/Library/BaseSortLib/BaseSortLib.inf > > HiiLib|MdeModulePkg/Library/UefiHiiLib/UefiHiiLib.inf > > > UefiHiiServicesLib|MdeModulePkg/Library/UefiHiiServicesLib/UefiHiiServicesL > ib.inf > > > UefiBootManagerLib|MdeModulePkg/Library/UefiBootManagerLib/UefiBoot > ManagerLib.inf > > -- > 2.32.0 (Apple Git-132) -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#106225): https://edk2.groups.io/g/devel/message/106225 Mute This Topic: https://groups.io/mt/99664829/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-