Re: [edk2-devel][Patch 0/3] Add destructor to CloseEvent

["Liming Gao" <liming.gao@intel.com>]

Thanks for your fix. The patch set is good. 

Reviewed-by: Liming Gao <liming.gao@intel.com>

>-----Original Message-----
>From: Xu, Wei6
>Sent: Friday, July 26, 2019 11:11 AM
>To: devel@edk2.groups.io
>Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
><liming.gao@intel.com>
>Subject: [edk2-devel][Patch 0/3] Add destructor to CloseEvent
>
>REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2012
>
>This patch set adds missing destructor for following libraries:
>MdePkg\Library\UefiDebugLibStdErr
>MdePkg\Library\UefiDebugLibConOut
>MdePkg\Library\UefiDebugLibDebugPortProtocol
>
>When driver is unloaded, the ExitBootSerivesEvent must be closed at
>the same time. Otherwise exception will occur when ExitBootServices.
>
>Cc: Michael D Kinney <michael.d.kinney@intel.com>
>Cc: Liming Gao <liming.gao@intel.com>
>
>Wei6 Xu (3):
>  MdePkg/UefiDebugLibConOut: Add destructor to CloseEvent
>  MdePkg/UefiDebugLibDebugPortProtocol: Add destructor to CloseEvent
>  MdePkg/UefiDebugLibStdErr: Add destructor to CloseEvent
>
> .../UefiDebugLibConOut/DebugLibConstructor.c       | 23
>++++++++++++++++++++++
> .../UefiDebugLibConOut/UefiDebugLibConOut.inf      |  1 +
> .../DebugLibConstructor.c                          | 23 ++++++++++++++++++++++
> .../UefiDebugLibDebugPortProtocol.inf              |  1 +
> .../UefiDebugLibStdErr/DebugLibConstructor.c       | 23
>++++++++++++++++++++++
> .../UefiDebugLibStdErr/UefiDebugLibStdErr.inf      |  1 +
> 6 files changed, 72 insertions(+)
>
>--
>2.16.2.windows.1