Re: [PATCH v3 0/3] Remove unnecessary NULL pointer check.

["Wu, Hao A" <hao.a.wu@intel.com>]

> -----Original Message-----
> From: Wu, Jiaxin
> Sent: Friday, January 18, 2019 3:18 PM
> To: edk2-devel@lists.01.org
> Cc: Ye, Ting; Fu, Siyuan; Wu, Hao A; Gao, Liming; Wu
> Subject: [PATCH v3 0/3] Remove unnecessary NULL pointer check.
> 
> v3: Updated the first patch to check the instance token.
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1469
> 
> Since the value retrieved from the list Entry can't be the NULL
> pointer, the unnecessary check can be removed.

The series is good to me:
Reviewed-by: Hao Wu <hao.a.wu@intel.com>

Best Regards,
Hao Wu

> 
> Cc: Ye Ting <ting.ye@intel.com>
> Cc: Fu Siyuan <siyuan.fu@intel.com>
> Cc: Wu Hao A <hao.a.wu@intel.com>
> Cc: Gao Liming <liming.gao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com
> 
> Jiaxin Wu (3):
>   MdeModulePkg/Dhcp4Dxe: Remove unnecessary NULL pointer check.
>   NetworkPkg/IScsiDxe: Remove unnecessary NULL pointer check.
>   NetworkPkg/DnsDxe: Remove unnecessary NULL pointer check.
> 
>  MdeModulePkg/Universal/Network/Dhcp4Dxe/Dhcp4Io.c      | 11 ++++------
>  NetworkPkg/DnsDxe/DnsDriver.c                          | 22 ++++++++-----------
>  NetworkPkg/IScsiDxe/IScsiConfig.c                      |  6 +----
>  3 files changed, 14 insertions(+), 25 deletions(-)
> 
> --
> 2.17.1.windows.2