Re: [edk2-devel] [PATCH v1 1/1] CryptoPkg/TlsLibNull: Remove MU_CHANGE comment markers

["Yao, Jiewen" <jiewen.yao@intel.com>]

Ah, thanks for the reminder. I overlooked this one.

Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Michael
> Kubacki
> Sent: Thursday, April 14, 2022 10:19 PM
> To: devel@edk2.groups.io
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J <jian.j.wang@intel.com>;
> Lu, Xiaoyu1 <xiaoyu1.lu@intel.com>; Jiang, Guomin <guomin.jiang@intel.com>
> Subject: Re: [edk2-devel] [PATCH v1 1/1] CryptoPkg/TlsLibNull: Remove
> MU_CHANGE comment markers
> 
> Please review this patch when possible. It is just removing two
> irrelevant comments.
> 
> Thanks,
> Michael
> 
> On 4/5/2022 8:50 PM, Michael Kubacki wrote:
> > From: Michael Kubacki <michael.kubacki@microsoft.com>
> >
> > MU_CHANGE tags are used in a different project (Project Mu) to
> > highlight deviations from Tianocore. Therefore, the comments are
> > not needed when the changes are present in a Tianocore repository.
> >
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Cc: Jian J Wang <jian.j.wang@intel.com>
> > Cc: Xiaoyu Lu <xiaoyu1.lu@intel.com>
> > Cc: Guomin Jiang <guomin.jiang@intel.com>
> > Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
> > ---
> >   CryptoPkg/Library/TlsLibNull/TlsConfigNull.c | 4 ----
> >   1 file changed, 4 deletions(-)
> >
> > diff --git a/CryptoPkg/Library/TlsLibNull/TlsConfigNull.c
> b/CryptoPkg/Library/TlsLibNull/TlsConfigNull.c
> > index b5d9aca06535..03726fd7264c 100644
> > --- a/CryptoPkg/Library/TlsLibNull/TlsConfigNull.c
> > +++ b/CryptoPkg/Library/TlsLibNull/TlsConfigNull.c
> > @@ -130,8 +130,6 @@ TlsSetVerify (
> >     ASSERT (FALSE);
> >   }
> >
> > -// MU_CHANGE - Proposed fixes for TCBZ960, invalid domain name (CN)
> accepted. [BEGIN]
> > -
> >   /**
> >     Set the specified host name to be verified.
> >
> > @@ -156,8 +154,6 @@ TlsSetVerifyHost (
> >     return EFI_UNSUPPORTED;
> >   }
> >
> > -// MU_CHANGE - Proposed fixes for TCBZ960, invalid domain name (CN)
> accepted. [END]
> > -
> >   /**
> >     Sets a TLS/SSL session ID to be used during TLS/SSL connect.
> >
> 
> 
> 
>