Re: [edk2] [PATCH] MdeModulePkg: Remove superfluous return statements

2017-01-19 Thread Zeng, Star
Star <star.z...@intel.com> Subject: RE: [edk2] [PATCH] MdeModulePkg: Remove superfluous return statements No need to resend path, I can help push it with the typo fixed. Thanks, Star -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Thomas

Re: [edk2] [PATCH] MdeModulePkg: Remove superfluous return statements

2017-01-19 Thread Zeng, Star
: Tian, Feng <feng.t...@intel.com> Subject: Re: [edk2] [PATCH] MdeModulePkg: Remove superfluous return statements On 19.01.2017 11:22, Zeng, Star wrote: > With the typo "explicitely" fixed in the commit log, Oh, right, stupid me ;-) Could you maybe fix that when picking up

Re: [edk2] [PATCH] MdeModulePkg: Remove superfluous return statements

2017-01-19 Thread Thomas Huth
eng.t...@intel.com>; Zeng, Star <star.z...@intel.com> > Subject: [edk2] [PATCH] MdeModulePkg: Remove superfluous return statements > > If the code eventually returns "Status" anyway, it does not make sense to > explicitely return "Status" in case of an

Re: [edk2] [PATCH] MdeModulePkg: Remove superfluous return statements

2017-01-19 Thread Zeng, Star
.@ml01.01.org Cc: Tian, Feng <feng.t...@intel.com>; Zeng, Star <star.z...@intel.com> Subject: [edk2] [PATCH] MdeModulePkg: Remove superfluous return statements If the code eventually returns "Status" anyway, it does not make sense to explicitely return "Status" in cas

[edk2] [PATCH] MdeModulePkg: Remove superfluous return statements

2017-01-19 Thread Thomas Huth
If the code eventually returns "Status" anyway, it does not make sense to explicitely return "Status" in case of an error, too. (this patch has been created with Coccinelle) Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Thomas Huth ---