[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]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to