From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:4864:20::343; helo=mail-wm1-x343.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wm1-x343.google.com (mail-wm1-x343.google.com [IPv6:2a00:1450:4864:20::343]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 53611211A1E2C for ; Fri, 14 Dec 2018 03:18:46 -0800 (PST) Received: by mail-wm1-x343.google.com with SMTP id a62so5311268wmh.4 for ; Fri, 14 Dec 2018 03:18:46 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=J905Kp836dm51to9cdbFc36x6iOE3H9gCJ5zFXsEkBM=; b=ajPHFoZLwYpD8Um8Q1Mgv7i8cHfn6gRqX981Uwb6dq1ZIv8dHLdTd06Gl269pxF0pX Iorpa9SOC7t1Czt4njzCQvClgyBcO19IYl7UFm0sDlCGExHqb15FqtXMrVAaISYcxyOW 75cJfGt5LYJiDGyDGY3UH63On3cQu8j1JSY8o= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=J905Kp836dm51to9cdbFc36x6iOE3H9gCJ5zFXsEkBM=; b=HKC12kaOuF6nVLm7Urqbun89okPtw6bW/uhgJ92fx0Qq/CyqLL2GDN1dMFR5bzsViU cacajZyVAReIWoGGlmUoZ8Cn/UNenoJGjCOfvI5miWM7euFjuYtbyuZnIG0Mt29Lb8SG jqIP5VsxrGQqqieSn3wtgN6SLEyo7OB9fkxvtSyo/0YRF9FoBvupXrBsDc9+1HkqTP4J RBT+7VvY6rdwJKB9U8d+mXUiRTXgHs8G2V0FcGbBCC3szdNcLlOs/C5MSP7YNXG8bJor puZoDQhZJLp0OlYNUC3GQD1gQ18Pj41G0KYZkW5f4QK7fm7WklY42JuKXiy2NFb66t15 wt4g== X-Gm-Message-State: AA+aEWb8Lu9IcP/GnscwoDeZXUsRcgLgVxf+OwhgftbCz8cPDoJJJwAU msB9hzt3b2VidcvMz2rPXphnWtCCuuM= X-Google-Smtp-Source: AFSGD/XSkEQivPcZMtUCrd91YohEeay9a4CjNGCV3ce1PtV7mVUQIxRD0hBB2vK0JtiiEr0HOj934A== X-Received: by 2002:a7b:c399:: with SMTP id s25mr3011068wmj.90.1544786325411; Fri, 14 Dec 2018 03:18:45 -0800 (PST) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id y138sm4862681wmc.16.2018.12.14.03.18.44 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 14 Dec 2018 03:18:44 -0800 (PST) Date: Fri, 14 Dec 2018 11:18:43 +0000 From: Leif Lindholm To: Shenglei Zhang Cc: edk2-devel@lists.01.org, Laszlo Ersek , Michael D Kinney , Liming Gao Message-ID: <20181214111843.367fwfzjdzzn7hzt@bivouac.eciton.net> References: <20181214082028.26812-1-shenglei.zhang@intel.com> <20181214082028.26812-3-shenglei.zhang@intel.com> MIME-Version: 1.0 In-Reply-To: <20181214082028.26812-3-shenglei.zhang@intel.com> User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [PATCH v2 2/3] MdePkg/BaseLib: Introduce CharToUpper and AsciiToUpper publicly X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 14 Dec 2018 11:18:47 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Dec 14, 2018 at 04:20:27PM +0800, Shenglei Zhang wrote: > Introduce two public functions CharToUpper and AsciiToUpper. > They have the same functions as InternalCharToUpper and > InternalBaseLibAsciiToUpper.Considering the internal functions will > be removed,so directly I change their function names to the public ones'. > https://bugzilla.tianocore.org/show_bug.cgi?id=1369 > > v2:Update the commit message and title Patch revision information does not belong in the commit message. Please move underneath the --- line, or keep it in the cover letter please. I also think the function deletion should be moved to 3/3, where the declarations are deleted. This was what I meant by my feedback on v1. Regards, Leif > Cc: Leif Lindholm > Cc: Laszlo Ersek > Cc: Michael D Kinney > Cc: Liming Gao > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Shenglei Zhang > --- > MdePkg/Include/Library/BaseLib.h | 40 +++++++++++++++++++++++++++++ > MdePkg/Library/BaseLib/SafeString.c | 8 +++--- > MdePkg/Library/BaseLib/String.c | 35 +++++-------------------- > 3 files changed, 51 insertions(+), 32 deletions(-) > > diff --git a/MdePkg/Include/Library/BaseLib.h b/MdePkg/Include/Library/BaseLib.h > index 8cc086983d..b861d82287 100644 > --- a/MdePkg/Include/Library/BaseLib.h > +++ b/MdePkg/Include/Library/BaseLib.h > @@ -2720,6 +2720,46 @@ AsciiStrnToUnicodeStrS ( > OUT UINTN *DestinationLength > ); > > +/** > + Convert a Unicode character to upper case only if > + it maps to a valid small-case ASCII character. > + > + This internal function only deal with Unicode character > + which maps to a valid small-case ASCII character, i.e. > + L'a' to L'z'. For other Unicode character, the input character > + is returned directly. > + > + @param Char The character to convert. > + > + @retval LowerCharacter If the Char is with range L'a' to L'z'. > + @retval Unchanged Otherwise. > + > +**/ > +CHAR16 > +EFIAPI > +CharToUpper ( > + IN CHAR16 Char > + ); > + > +/** > + Converts a lowercase Ascii character to upper one. > + > + If Chr is lowercase Ascii character, then converts it to upper one. > + > + If Value >= 0xA0, then ASSERT(). > + If (Value & 0x0F) >= 0x0A, then ASSERT(). > + > + @param Chr one Ascii character > + > + @return The uppercase value of Ascii character > + > +**/ > +CHAR8 > +EFIAPI > +AsciiToUpper ( > + IN CHAR8 Chr > + ); > + > /** > Converts an 8-bit value to an 8-bit BCD value. > > diff --git a/MdePkg/Library/BaseLib/SafeString.c b/MdePkg/Library/BaseLib/SafeString.c > index 417497cbc9..17f88b46d8 100644 > --- a/MdePkg/Library/BaseLib/SafeString.c > +++ b/MdePkg/Library/BaseLib/SafeString.c > @@ -905,7 +905,7 @@ StrHexToUintnS ( > String++; > } > > - if (InternalCharToUpper (*String) == L'X') { > + if (CharToUpper (*String) == L'X') { > if (*(String - 1) != L'0') { > *Data = 0; > return RETURN_SUCCESS; > @@ -1036,7 +1036,7 @@ StrHexToUint64S ( > String++; > } > > - if (InternalCharToUpper (*String) == L'X') { > + if (CharToUpper (*String) == L'X') { > if (*(String - 1) != L'0') { > *Data = 0; > return RETURN_SUCCESS; > @@ -2459,7 +2459,7 @@ AsciiStrHexToUintnS ( > String++; > } > > - if (InternalBaseLibAsciiToUpper (*String) == 'X') { > + if (AsciiToUpper (*String) == 'X') { > if (*(String - 1) != '0') { > *Data = 0; > return RETURN_SUCCESS; > @@ -2586,7 +2586,7 @@ AsciiStrHexToUint64S ( > String++; > } > > - if (InternalBaseLibAsciiToUpper (*String) == 'X') { > + if (AsciiToUpper (*String) == 'X') { > if (*(String - 1) != '0') { > *Data = 0; > return RETURN_SUCCESS; > diff --git a/MdePkg/Library/BaseLib/String.c b/MdePkg/Library/BaseLib/String.c > index e6df12797d..ced1b3f975 100644 > --- a/MdePkg/Library/BaseLib/String.c > +++ b/MdePkg/Library/BaseLib/String.c > @@ -552,7 +552,7 @@ InternalIsDecimalDigitCharacter ( > **/ > CHAR16 > EFIAPI > -InternalCharToUpper ( > +CharToUpper ( > IN CHAR16 Char > ) > { > @@ -586,7 +586,7 @@ InternalHexCharToUintn ( > return Char - L'0'; > } > > - return (10 + InternalCharToUpper (Char) - L'A'); > + return (10 + CharToUpper (Char) - L'A'); > } > > /** > @@ -1166,27 +1166,6 @@ AsciiStrCmp ( > return *FirstString - *SecondString; > } > > -/** > - Converts a lowercase Ascii character to upper one. > - > - If Chr is lowercase Ascii character, then converts it to upper one. > - > - If Value >= 0xA0, then ASSERT(). > - If (Value & 0x0F) >= 0x0A, then ASSERT(). > - > - @param Chr one Ascii character > - > - @return The uppercase value of Ascii character > - > -**/ > -CHAR8 > -EFIAPI > -InternalBaseLibAsciiToUpper ( > - IN CHAR8 Chr > - ) > -{ > - return (UINT8) ((Chr >= 'a' && Chr <= 'z') ? Chr - ('a' - 'A') : Chr); > -} > > /** > Convert a ASCII character to numerical value. > @@ -1211,7 +1190,7 @@ InternalAsciiHexCharToUintn ( > return Char - '0'; > } > > - return (10 + InternalBaseLibAsciiToUpper (Char) - 'A'); > + return (10 + AsciiToUpper (Char) - 'A'); > } > > > @@ -1260,13 +1239,13 @@ AsciiStriCmp ( > ASSERT (AsciiStrSize (FirstString)); > ASSERT (AsciiStrSize (SecondString)); > > - UpperFirstString = InternalBaseLibAsciiToUpper (*FirstString); > - UpperSecondString = InternalBaseLibAsciiToUpper (*SecondString); > + UpperFirstString = AsciiToUpper (*FirstString); > + UpperSecondString = AsciiToUpper (*SecondString); > while ((*FirstString != '\0') && (*SecondString != '\0') && (UpperFirstString == UpperSecondString)) { > FirstString++; > SecondString++; > - UpperFirstString = InternalBaseLibAsciiToUpper (*FirstString); > - UpperSecondString = InternalBaseLibAsciiToUpper (*SecondString); > + UpperFirstString = AsciiToUpper (*FirstString); > + UpperSecondString = AsciiToUpper (*SecondString); > } > > return UpperFirstString - UpperSecondString; > -- > 2.18.0.windows.1 >