From: "Wu, Hao A" <hao.a.wu@intel.com>
To: "Zhang, Shenglei" <shenglei.zhang@intel.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Wang, Jian J" <jian.j.wang@intel.com>,
"Ni, Ray" <ray.ni@intel.com>, "Zeng, Star" <star.zeng@intel.com>
Subject: Re: [PATCH v5 1/7] MdeModulePkg/EbcDebugger: Change function names
Date: Wed, 30 Jan 2019 08:04:51 +0000 [thread overview]
Message-ID: <B80AF82E9BFB8E4FBD8C89DA810C6A093C8927FC@SHSMSX104.ccr.corp.intel.com> (raw)
In-Reply-To: <20190130063417.11308-2-shenglei.zhang@intel.com>
> -----Original Message-----
> From: Zhang, Shenglei
> Sent: Wednesday, January 30, 2019 2:34 PM
> To: edk2-devel@lists.01.org
> Cc: Wang, Jian J; Wu, Hao A; Ni, Ray; Zeng, Star
> Subject: [PATCH v5 1/7] MdeModulePkg/EbcDebugger: Change function names
>
> Change UnicodeToUpper to InternalUnicodeToUpper.
> Change AsciiToUpper to InternalAsciiToUpper.
> These changes are committed for bisectability, or build failure will
> occur when 2/7 is applied. Because the introduced function names in
> 2/7 are the same as ones in EdbSupporting.c.
> https://bugzilla.tianocore.org/show_bug.cgi?id=1369
>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao Wu <hao.a.wu@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
> Reviewed-by: Hao Wu <hao.a.wu@intel.com>
> Reviewed-by: Liming Gao <liming.gao@intel.com>
> ---
> .../Universal/EbcDxe/EbcDebugger/EdbSupportString.c | 12 ++++++------
Please help to update the copyright year for file EdbSupportString.c.
You can keep my 'Reviewed-by' tag and do not need to send a new version of
series for this.
Best Regards,
Hao Wu
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSupportString.c
> b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSupportString.c
> index 78a0559079..02e1876ffa 100644
> --- a/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSupportString.c
> +++ b/MdeModulePkg/Universal/EbcDxe/EbcDebugger/EdbSupportString.c
> @@ -316,7 +316,7 @@ AsciiAtoi (
> **/
> STATIC
> CHAR16
> -UnicodeToUpper (
> +InternalUnicodeToUpper (
> IN CHAR16 Chr
> )
> {
> @@ -332,7 +332,7 @@ UnicodeToUpper (
> **/
> STATIC
> CHAR8
> -AsciiToUpper (
> +InternalAsciiToUpper (
> IN CHAR8 Chr
> )
> {
> @@ -390,12 +390,12 @@ StriCmp (
> )
> {
> while ((*String != L'\0') &&
> - (UnicodeToUpper (*String) == UnicodeToUpper (*String2))) {
> + (InternalUnicodeToUpper (*String) == InternalUnicodeToUpper
> (*String2))) {
> String++;
> String2++;
> }
>
> - return UnicodeToUpper (*String) - UnicodeToUpper (*String2);
> + return InternalUnicodeToUpper (*String) - InternalUnicodeToUpper
> (*String2);
> }
>
> /**
> @@ -418,12 +418,12 @@ StriCmpUnicodeAndAscii (
> )
> {
> while ((*String != L'\0') &&
> - (UnicodeToUpper (*String) == (CHAR16)AsciiToUpper (*String2))) {
> + (InternalUnicodeToUpper (*String) == (CHAR16)InternalAsciiToUpper
> (*String2))) {
> String++;
> String2++;
> }
>
> - return UnicodeToUpper (*String) - (CHAR16)AsciiToUpper (*String2);
> + return InternalUnicodeToUpper (*String) - (CHAR16)InternalAsciiToUpper
> (*String2);
> }
>
> /**
> --
> 2.18.0.windows.1
next prev parent reply other threads:[~2019-01-30 8:04 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-30 6:34 [PATCH v5 0/7] Introduce two public functions and remove internal ones Shenglei Zhang
2019-01-30 6:34 ` [PATCH v5 1/7] MdeModulePkg/EbcDebugger: Change function names Shenglei Zhang
2019-01-30 8:04 ` Wu, Hao A [this message]
2019-01-30 6:34 ` [PATCH v5 2/7] MdePkg/BaseLib: Introduce CharToUpper and AsciiCharToUpper publicly Shenglei Zhang
2019-01-30 6:34 ` [PATCH v5 3/7] MdePkg/BaseLib: Remove definitions of two functions Shenglei Zhang
2019-01-30 6:34 ` [PATCH v5 4/7] MdeModulePkg/EbcDebugger: Use AsciiCharToUpper and CharToUpper Shenglei Zhang
2019-01-30 8:04 ` Wu, Hao A
2019-01-30 6:34 ` [PATCH v5 5/7] MdeModulePkg/DxeHttpLib: Use BaseLib api AsciiCharToUpper Shenglei Zhang
2019-01-30 8:04 ` Wu, Hao A
2019-01-30 6:34 ` [PATCH v5 6/7] ShellPkg/Shell: Use BaseLib api CharToUpper Shenglei Zhang
2019-01-30 16:13 ` Carsey, Jaben
2019-01-30 6:34 ` [PATCH v5 7/7] ShellPkg/UefiShellLib: " Shenglei Zhang
2019-01-30 16:13 ` Carsey, Jaben
2019-01-30 9:36 ` [PATCH v5 0/7] Introduce two public functions and remove internal ones Ni, Ruiyu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=B80AF82E9BFB8E4FBD8C89DA810C6A093C8927FC@SHSMSX104.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox