Reviewed-by: Wu Jiaxin <[email protected]> Thanks, Jiaxin
> -----Original Message----- > From: edk2-devel [mailto:[email protected]] On Behalf Of > Thomas Huth > Sent: Thursday, January 19, 2017 5:38 PM > To: [email protected] > Cc: Fu, Siyuan <[email protected]>; Wu, Jiaxin <[email protected]> > Subject: [edk2] [PATCH] NetworkPkg: Remove superfluous return statement. > > If the code eventually returns "Status" anyway, it does not make > sense to explicitely return "Status" in case of an error, too. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Thomas Huth <[email protected]> > --- > NetworkPkg/HttpDxe/HttpsSupport.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/NetworkPkg/HttpDxe/HttpsSupport.c > b/NetworkPkg/HttpDxe/HttpsSupport.c > index 77e5371..f0077dd 100644 > --- a/NetworkPkg/HttpDxe/HttpsSupport.c > +++ b/NetworkPkg/HttpDxe/HttpsSupport.c > @@ -1294,10 +1294,6 @@ TlsCloseSession ( > FreePool (BufferOut); > NetbufFree (PacketOut); > > - if (EFI_ERROR (Status)) { > - return Status; > - } > - > return Status; > } > > -- > 1.8.3.1 > > _______________________________________________ > edk2-devel mailing list > [email protected] > https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
