Thanks a lot.

Just one minor comment, in RemoveTrailingSpaces(), could you help to refine:
  String[Length] = 0;
to:
  String[Length] = '\0';

Also, could you help to send a V2 patch (the change made in the below pull 
request) to the mailing list?
(Mainly step 12 in 
https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Development-Process#the-developer-process-for-the-edk-ii-project)
I will follow up with the code reviewing and merging. Thanks in advance.

Best Regards,
Hao Wu

From: gopic via groups.io <gopic=ami....@groups.io>
Sent: Wednesday, October 26, 2022 9:44 PM
To: Wu; Wu, Hao A <hao.a...@intel.com>; devel@edk2.groups.io
Subject: Re: [edk2-devel] [PATCH] MdeModulePkg: Improved ScsiDiskDxe driver 
updates ControllerNameTable with common string SCSI Disk Device for all SCSI 
disk

created pull request.
Pull request link: MdeModulePkg/ScsiDiskDxe: Update proper device name for 
ScsiDisk drives by GopiCheripally · Pull Request #3537 · tianocore/edk2 
(github.com)<https://github.com/tianocore/edk2/pull/3537>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#95628): https://edk2.groups.io/g/devel/message/95628
Mute This Topic: https://groups.io/mt/94399351/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to