Re: [edk2] [patch V2] MdeModulePkg/ReportStatusCodeLib: Avoid using AllocatePool if possible

2019-02-17 Thread Bi, Dandan
> -Original Message- > From: Zeng, Star > Sent: Friday, February 15, 2019 5:25 PM > To: Bi, Dandan ; edk2-devel@lists.01.org > Cc: Wu, Hao A ; Michael Turner > ; Max Knutsen > ; Gao, Liming ; Zeng, > Star > Subject: RE: [edk2] [patch V2] MdeModulePkg/ReportS

Re: [edk2] [patch V2] MdeModulePkg/ReportStatusCodeLib: Avoid using AllocatePool if possible

2019-02-15 Thread Gao, Liming
en ; Gao, > Liming ; Zeng, Star > Subject: RE: [edk2] [patch V2] MdeModulePkg/ReportStatusCodeLib: Avoid using > AllocatePool if possible > > After applying the patch, the author is Max and the Signed-off-by is Dandan, > is it expected? > > And the code below is better t

Re: [edk2] [patch V2] MdeModulePkg/ReportStatusCodeLib: Avoid using AllocatePool if possible

2019-02-15 Thread Gao, Liming
: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > > Dandan Bi > > Sent: Thursday, February 14, 2019 10:39 AM > > To: edk2-devel@lists.01.org > > Cc: Wu, Hao A ; Michael Turner > > ; Max Knutsen > > ; Gao, Liming > > Subject: [edk2] [patch

Re: [edk2] [patch V2] MdeModulePkg/ReportStatusCodeLib: Avoid using AllocatePool if possible

2019-02-15 Thread Zeng, Star
vel@lists.01.org Cc: Wu, Hao A ; Michael Turner ; Max Knutsen ; Gao, Liming Subject: [edk2] [patch V2] MdeModulePkg/ReportStatusCodeLib: Avoid using AllocatePool if possible From: Max Knutsen REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1114 V2: simplify the code l

Re: [edk2] [patch V2] MdeModulePkg/ReportStatusCodeLib: Avoid using AllocatePool if possible

2019-02-13 Thread Bi, Dandan
019 10:39 AM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A ; Michael Turner > ; Max Knutsen > ; Gao, Liming > Subject: [edk2] [patch V2] MdeModulePkg/ReportStatusCodeLib: Avoid > using AllocatePool if possible > > From: Max Knutsen > > REF: https://bugzilla.tianocore.o

[edk2] [patch V2] MdeModulePkg/ReportStatusCodeLib: Avoid using AllocatePool if possible

2019-02-13 Thread Dandan Bi
From: Max Knutsen REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1114 V2: simplify the code logic. update if (!mHaveExitedBootServices && (StatusCodeData != (EFI_STATUS_CODE_DATA *) StatusCodeBuffer)) { gBS->FreePool (StatusCodeData); } to if (StatusCodeData != (EFI_STATUS_CODE_DATA