* [PATCH 0/2] Add two public functions
@ 2018-12-13 8:34 Shenglei Zhang
2018-12-13 8:34 ` [PATCH 1/2] MdePkg/BaseLib: " Shenglei Zhang
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Shenglei Zhang @ 2018-12-13 8:34 UTC (permalink / raw)
To: edk2-devel; +Cc: Michael D Kinney, Liming Gao, Bob Feng, Yonghong Zhu
Add two public functions,CharToUpper and AsciiToUpper,and
remove the same functional functions,InternalCharToUpper
and InternalBaseLibAsciiToUpper.
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Shenglei Zhang (2):
MdePkg/BaseLib: Add two public functions
BaseTools/Common: Remove InternalCharToUpper
BaseTools/Source/C/Common/CommonLib.c | 16 ++-------
BaseTools/Source/C/Common/CommonLib.h | 4 ---
MdePkg/Include/Library/BaseLib.h | 40 +++++++++++++++++++++
MdePkg/Library/BaseLib/BaseLibInternals.h | 42 -----------------------
MdePkg/Library/BaseLib/SafeString.c | 8 ++---
MdePkg/Library/BaseLib/String.c | 35 ++++---------------
6 files changed, 53 insertions(+), 92 deletions(-)
--
2.18.0.windows.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] MdePkg/BaseLib: Add two public functions
2018-12-13 8:34 [PATCH 0/2] Add two public functions Shenglei Zhang
@ 2018-12-13 8:34 ` Shenglei Zhang
2018-12-13 10:33 ` Laszlo Ersek
2018-12-13 10:49 ` Leif Lindholm
2018-12-13 8:34 ` [PATCH 2/2] BaseTools/Common: Remove InternalCharToUpper Shenglei Zhang
2018-12-13 10:51 ` [PATCH 0/2] Add two public functions Leif Lindholm
2 siblings, 2 replies; 7+ messages in thread
From: Shenglei Zhang @ 2018-12-13 8:34 UTC (permalink / raw)
To: edk2-devel; +Cc: Michael D Kinney, Liming Gao
Add two public functions, CharToUpper and AsciiToUpper.
InternalCharToUpper and InternalBaseLibAsciiToUpper have the same
functions as CharToUpper and AsciiToUpper, but they are internal ones.
So the internal ones are removed and replace them with public ones
in other places.
https://bugzilla.tianocore.org/show_bug.cgi?id=1369
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
---
MdePkg/Include/Library/BaseLib.h | 40 +++++++++++++++++++++
MdePkg/Library/BaseLib/BaseLibInternals.h | 42 -----------------------
MdePkg/Library/BaseLib/SafeString.c | 8 ++---
MdePkg/Library/BaseLib/String.c | 35 ++++---------------
4 files changed, 51 insertions(+), 74 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/BaseLibInternals.h b/MdePkg/Library/BaseLib/BaseLibInternals.h
index 8855231c1a..9db925b157 100644
--- a/MdePkg/Library/BaseLib/BaseLibInternals.h
+++ b/MdePkg/Library/BaseLib/BaseLibInternals.h
@@ -469,28 +469,6 @@ InternalIsDecimalDigitCharacter (
);
-/**
- 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
-InternalCharToUpper (
- IN CHAR16 Char
- );
-
-
/**
Convert a Unicode character to numerical value.
@@ -552,26 +530,6 @@ InternalAsciiIsDecimalDigitCharacter (
);
-/**
- 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
- );
-
-
/**
Check if a ASCII character is a hexadecimal character.
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
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] BaseTools/Common: Remove InternalCharToUpper
2018-12-13 8:34 [PATCH 0/2] Add two public functions Shenglei Zhang
2018-12-13 8:34 ` [PATCH 1/2] MdePkg/BaseLib: " Shenglei Zhang
@ 2018-12-13 8:34 ` Shenglei Zhang
2018-12-13 10:51 ` [PATCH 0/2] Add two public functions Leif Lindholm
2 siblings, 0 replies; 7+ messages in thread
From: Shenglei Zhang @ 2018-12-13 8:34 UTC (permalink / raw)
To: edk2-devel; +Cc: Bob Feng, Liming Gao, Yonghong Zhu
InternalCharToUpper is an internal function.
So remove InternalCharToUpper and replace it with a public
function CharToUpper which has the same function as the internal one
in all places.
https://bugzilla.tianocore.org/show_bug.cgi?id=1369
Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Yonghong Zhu <yonghong.zhu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
---
BaseTools/Source/C/Common/CommonLib.c | 16 ++--------------
BaseTools/Source/C/Common/CommonLib.h | 4 ----
2 files changed, 2 insertions(+), 18 deletions(-)
diff --git a/BaseTools/Source/C/Common/CommonLib.c b/BaseTools/Source/C/Common/CommonLib.c
index 5c40fdb5fd..878b593a4e 100644
--- a/BaseTools/Source/C/Common/CommonLib.c
+++ b/BaseTools/Source/C/Common/CommonLib.c
@@ -740,18 +740,6 @@ Returns:
#endif
}
-CHAR16
-InternalCharToUpper (
- CHAR16 Char
- )
-{
- if (Char >= L'a' && Char <= L'z') {
- return (CHAR16) (Char - (L'a' - L'A'));
- }
-
- return Char;
-}
-
UINTN
StrnLenS (
CONST CHAR16 *String,
@@ -1089,7 +1077,7 @@ StrHexToUint64S (
String++;
}
- if (InternalCharToUpper (*String) == L'X') {
+ if (CharToUpper (*String) == L'X') {
if (*(String - 1) != L'0') {
*Data = 0;
return RETURN_SUCCESS;
@@ -1264,7 +1252,7 @@ InternalHexCharToUintn (
return Char - L'0';
}
- return (10 + InternalCharToUpper (Char) - L'A');
+ return (10 + CharToUpper (Char) - L'A');
}
diff --git a/BaseTools/Source/C/Common/CommonLib.h b/BaseTools/Source/C/Common/CommonLib.h
index 4e1541bc70..b81584c7d4 100644
--- a/BaseTools/Source/C/Common/CommonLib.h
+++ b/BaseTools/Source/C/Common/CommonLib.h
@@ -304,10 +304,6 @@ StrnLenS (
UINTN MaxSize
);
-CHAR16
-InternalCharToUpper (
- CHAR16 Char
- );
INTN
StrCmp (
--
2.18.0.windows.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] MdePkg/BaseLib: Add two public functions
2018-12-13 8:34 ` [PATCH 1/2] MdePkg/BaseLib: " Shenglei Zhang
@ 2018-12-13 10:33 ` Laszlo Ersek
2018-12-13 10:49 ` Leif Lindholm
1 sibling, 0 replies; 7+ messages in thread
From: Laszlo Ersek @ 2018-12-13 10:33 UTC (permalink / raw)
To: Shenglei Zhang, edk2-devel; +Cc: Michael D Kinney, Liming Gao
On 12/13/18 09:34, Shenglei Zhang wrote:
> Add two public functions, CharToUpper and AsciiToUpper.
> InternalCharToUpper and InternalBaseLibAsciiToUpper have the same
> functions as CharToUpper and AsciiToUpper, but they are internal ones.
> So the internal ones are removed and replace them with public ones
> in other places.
> https://bugzilla.tianocore.org/show_bug.cgi?id=1369
>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
> ---
> MdePkg/Include/Library/BaseLib.h | 40 +++++++++++++++++++++
> MdePkg/Library/BaseLib/BaseLibInternals.h | 42 -----------------------
> MdePkg/Library/BaseLib/SafeString.c | 8 ++---
> MdePkg/Library/BaseLib/String.c | 35 ++++---------------
> 4 files changed, 51 insertions(+), 74 deletions(-)
I have no comment on the code, but please change the subject line to:
MdePkg/BaseLib: introduce CharToUpper() and AsciiToUpper() publicly
or something similar.
Thanks,
Laszlo
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] MdePkg/BaseLib: Add two public functions
2018-12-13 8:34 ` [PATCH 1/2] MdePkg/BaseLib: " Shenglei Zhang
2018-12-13 10:33 ` Laszlo Ersek
@ 2018-12-13 10:49 ` Leif Lindholm
1 sibling, 0 replies; 7+ messages in thread
From: Leif Lindholm @ 2018-12-13 10:49 UTC (permalink / raw)
To: Shenglei Zhang; +Cc: edk2-devel, Michael D Kinney, Liming Gao
Hi Zhenglei,
Please change the title to
MdePkg/BaseLib: add common versions of CharToUpper and AsciiToUpper
and split the change/deletion in String/SafeString/BaseLibInternals
into a separate patch, following after this one.
Other than that, glad to see this consolidation.
Regards,
Leif
On Thu, Dec 13, 2018 at 04:34:38PM +0800, Shenglei Zhang wrote:
> Add two public functions, CharToUpper and AsciiToUpper.
> InternalCharToUpper and InternalBaseLibAsciiToUpper have the same
> functions as CharToUpper and AsciiToUpper, but they are internal ones.
> So the internal ones are removed and replace them with public ones
> in other places.
> https://bugzilla.tianocore.org/show_bug.cgi?id=1369
>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Shenglei Zhang <shenglei.zhang@intel.com>
> ---
> MdePkg/Include/Library/BaseLib.h | 40 +++++++++++++++++++++
> MdePkg/Library/BaseLib/BaseLibInternals.h | 42 -----------------------
> MdePkg/Library/BaseLib/SafeString.c | 8 ++---
> MdePkg/Library/BaseLib/String.c | 35 ++++---------------
> 4 files changed, 51 insertions(+), 74 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/BaseLibInternals.h b/MdePkg/Library/BaseLib/BaseLibInternals.h
> index 8855231c1a..9db925b157 100644
> --- a/MdePkg/Library/BaseLib/BaseLibInternals.h
> +++ b/MdePkg/Library/BaseLib/BaseLibInternals.h
> @@ -469,28 +469,6 @@ InternalIsDecimalDigitCharacter (
> );
>
>
> -/**
> - 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
> -InternalCharToUpper (
> - IN CHAR16 Char
> - );
> -
> -
> /**
> Convert a Unicode character to numerical value.
>
> @@ -552,26 +530,6 @@ InternalAsciiIsDecimalDigitCharacter (
> );
>
>
> -/**
> - 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
> - );
> -
> -
> /**
> Check if a ASCII character is a hexadecimal character.
>
> 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
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/2] Add two public functions
2018-12-13 8:34 [PATCH 0/2] Add two public functions Shenglei Zhang
2018-12-13 8:34 ` [PATCH 1/2] MdePkg/BaseLib: " Shenglei Zhang
2018-12-13 8:34 ` [PATCH 2/2] BaseTools/Common: Remove InternalCharToUpper Shenglei Zhang
@ 2018-12-13 10:51 ` Leif Lindholm
2018-12-14 6:17 ` Zhang, Shenglei
2 siblings, 1 reply; 7+ messages in thread
From: Leif Lindholm @ 2018-12-13 10:51 UTC (permalink / raw)
To: Shenglei Zhang; +Cc: edk2-devel, Michael D Kinney, Liming Gao
Please let the subject line give some sort of hint of what is being
done, and to what. "Add two functions" is not substantially more
descriptive than "add 572 characters".
In this case, your're moving previously internal string helper functions
to BaseLib.
On Thu, Dec 13, 2018 at 04:34:37PM +0800, Shenglei Zhang wrote:
> Add two public functions,CharToUpper and AsciiToUpper,and
> remove the same functional functions,InternalCharToUpper
> and InternalBaseLibAsciiToUpper.
>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Bob Feng <bob.c.feng@intel.com>
> Cc: Yonghong Zhu <yonghong.zhu@intel.com>
> Shenglei Zhang (2):
> MdePkg/BaseLib: Add two public functions
> BaseTools/Common: Remove InternalCharToUpper
>
> BaseTools/Source/C/Common/CommonLib.c | 16 ++-------
> BaseTools/Source/C/Common/CommonLib.h | 4 ---
> MdePkg/Include/Library/BaseLib.h | 40 +++++++++++++++++++++
> MdePkg/Library/BaseLib/BaseLibInternals.h | 42 -----------------------
> MdePkg/Library/BaseLib/SafeString.c | 8 ++---
> MdePkg/Library/BaseLib/String.c | 35 ++++---------------
> 6 files changed, 53 insertions(+), 92 deletions(-)
>
> --
> 2.18.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/2] Add two public functions
2018-12-13 10:51 ` [PATCH 0/2] Add two public functions Leif Lindholm
@ 2018-12-14 6:17 ` Zhang, Shenglei
0 siblings, 0 replies; 7+ messages in thread
From: Zhang, Shenglei @ 2018-12-14 6:17 UTC (permalink / raw)
To: Leif Lindholm; +Cc: edk2-devel@lists.01.org, Kinney, Michael D, Gao, Liming
Hi Leif
Thanks for your constructive guide. I 'll improve my later patches.
Thanks,
Shenglei
> -----Original Message-----
> From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
> Sent: Thursday, December 13, 2018 6:52 PM
> To: Zhang, Shenglei <shenglei.zhang@intel.com>
> Cc: edk2-devel@lists.01.org; Kinney, Michael D
> <michael.d.kinney@intel.com>; Gao, Liming <liming.gao@intel.com>
> Subject: Re: [edk2] [PATCH 0/2] Add two public functions
>
> Please let the subject line give some sort of hint of what is being
> done, and to what. "Add two functions" is not substantially more
> descriptive than "add 572 characters".
>
> In this case, your're moving previously internal string helper functions
> to BaseLib.
>
> On Thu, Dec 13, 2018 at 04:34:37PM +0800, Shenglei Zhang wrote:
> > Add two public functions,CharToUpper and AsciiToUpper,and
> > remove the same functional functions,InternalCharToUpper
> > and InternalBaseLibAsciiToUpper.
> >
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Liming Gao <liming.gao@intel.com>
> > Cc: Bob Feng <bob.c.feng@intel.com>
> > Cc: Yonghong Zhu <yonghong.zhu@intel.com>
> > Shenglei Zhang (2):
> > MdePkg/BaseLib: Add two public functions
> > BaseTools/Common: Remove InternalCharToUpper
> >
> > BaseTools/Source/C/Common/CommonLib.c | 16 ++-------
> > BaseTools/Source/C/Common/CommonLib.h | 4 ---
> > MdePkg/Include/Library/BaseLib.h | 40 +++++++++++++++++++++
> > MdePkg/Library/BaseLib/BaseLibInternals.h | 42 -----------------------
> > MdePkg/Library/BaseLib/SafeString.c | 8 ++---
> > MdePkg/Library/BaseLib/String.c | 35 ++++---------------
> > 6 files changed, 53 insertions(+), 92 deletions(-)
> >
> > --
> > 2.18.0.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-12-14 6:17 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-13 8:34 [PATCH 0/2] Add two public functions Shenglei Zhang
2018-12-13 8:34 ` [PATCH 1/2] MdePkg/BaseLib: " Shenglei Zhang
2018-12-13 10:33 ` Laszlo Ersek
2018-12-13 10:49 ` Leif Lindholm
2018-12-13 8:34 ` [PATCH 2/2] BaseTools/Common: Remove InternalCharToUpper Shenglei Zhang
2018-12-13 10:51 ` [PATCH 0/2] Add two public functions Leif Lindholm
2018-12-14 6:17 ` Zhang, Shenglei
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox