* [Patch] BaseTools CommonLib: Remove the unnecessary print message in PcdValueCommon
@ 2018-01-31 9:03 Liming Gao
2018-01-31 10:39 ` Zhu, Yonghong
0 siblings, 1 reply; 3+ messages in thread
From: Liming Gao @ 2018-01-31 9:03 UTC (permalink / raw)
To: edk2-devel
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Liming Gao <liming.gao@intel.com>
---
BaseTools/Source/C/Common/PcdValueCommon.c | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/BaseTools/Source/C/Common/PcdValueCommon.c b/BaseTools/Source/C/Common/PcdValueCommon.c
index 92328da..210f87b 100644
--- a/BaseTools/Source/C/Common/PcdValueCommon.c
+++ b/BaseTools/Source/C/Common/PcdValueCommon.c
@@ -330,9 +330,7 @@ Returns:
break;
case PcdDataTypePointer:
Value = &PcdList[Index].Value[1];
- printf ("Value = %s\n", PcdList[Index].Value);
for (*Size = 0, Byte = (UINT8) strtoul(Value, &End, 16); Value != End; Byte = (UINT8) strtoul(Value, &End, 16), *Size = *Size + 1) {
- printf("%x\n", Byte);
Value = End + 1;
}
Buffer = malloc(*Size + 1);
@@ -401,7 +399,6 @@ Returns:
PcdList[Index].Value = malloc(Size * 5 + 3);
PcdList[Index].Value[0] = '{';
for (ValueIndex = 0; ValueIndex < Size; ValueIndex++) {
- printf("Value[%d] = %02x\n", ValueIndex, Value[ValueIndex]);
sprintf(&PcdList[Index].Value[1 + ValueIndex * 5], "0x%02x,", Value[ValueIndex]);
}
PcdList[Index].Value[1 + Size * 5 - 1] = '}';
@@ -724,15 +721,11 @@ Returns:
if (*InputFileName == NULL) {
fprintf (stderr, "Missing option. Input files is not specified\n");
exit (EXIT_FAILURE);
- } else {
- printf ("Input file name is %s\n", *InputFileName);
}
if (*OutputFileName == NULL) {
fprintf (stderr, "Missing option. Output file is not specified\n");
exit (EXIT_FAILURE);
- } else {
- printf ("Output file name is %s\n", *OutputFileName);
}
}
@@ -761,7 +754,6 @@ Returns:
UINT8 *FileBuffer;
UINT32 FileSize;
- printf ("PCD tool start.\n");
InputFileName = NULL;
OutputFileName = NULL;
@@ -790,7 +782,5 @@ Returns:
//
WriteOutputFile (OutputFileName);
- printf ("PCD tool done.\n");
-
exit (EXIT_SUCCESS);
}
--
2.8.0.windows.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [Patch] BaseTools CommonLib: Remove the unnecessary print message in PcdValueCommon
2018-01-31 9:03 [Patch] BaseTools CommonLib: Remove the unnecessary print message in PcdValueCommon Liming Gao
@ 2018-01-31 10:39 ` Zhu, Yonghong
2018-02-01 8:18 ` Ard Biesheuvel
0 siblings, 1 reply; 3+ messages in thread
From: Zhu, Yonghong @ 2018-01-31 10:39 UTC (permalink / raw)
To: Gao, Liming, edk2-devel@lists.01.org
Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com>
Best Regards,
Zhu Yonghong
-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Liming Gao
Sent: Wednesday, January 31, 2018 5:04 PM
To: edk2-devel@lists.01.org
Subject: [edk2] [Patch] BaseTools CommonLib: Remove the unnecessary print message in PcdValueCommon
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Liming Gao <liming.gao@intel.com>
---
BaseTools/Source/C/Common/PcdValueCommon.c | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/BaseTools/Source/C/Common/PcdValueCommon.c b/BaseTools/Source/C/Common/PcdValueCommon.c
index 92328da..210f87b 100644
--- a/BaseTools/Source/C/Common/PcdValueCommon.c
+++ b/BaseTools/Source/C/Common/PcdValueCommon.c
@@ -330,9 +330,7 @@ Returns:
break;
case PcdDataTypePointer:
Value = &PcdList[Index].Value[1];
- printf ("Value = %s\n", PcdList[Index].Value);
for (*Size = 0, Byte = (UINT8) strtoul(Value, &End, 16); Value != End; Byte = (UINT8) strtoul(Value, &End, 16), *Size = *Size + 1) {
- printf("%x\n", Byte);
Value = End + 1;
}
Buffer = malloc(*Size + 1);
@@ -401,7 +399,6 @@ Returns:
PcdList[Index].Value = malloc(Size * 5 + 3);
PcdList[Index].Value[0] = '{';
for (ValueIndex = 0; ValueIndex < Size; ValueIndex++) {
- printf("Value[%d] = %02x\n", ValueIndex, Value[ValueIndex]);
sprintf(&PcdList[Index].Value[1 + ValueIndex * 5], "0x%02x,", Value[ValueIndex]);
}
PcdList[Index].Value[1 + Size * 5 - 1] = '}'; @@ -724,15 +721,11 @@ Returns:
if (*InputFileName == NULL) {
fprintf (stderr, "Missing option. Input files is not specified\n");
exit (EXIT_FAILURE);
- } else {
- printf ("Input file name is %s\n", *InputFileName);
}
if (*OutputFileName == NULL) {
fprintf (stderr, "Missing option. Output file is not specified\n");
exit (EXIT_FAILURE);
- } else {
- printf ("Output file name is %s\n", *OutputFileName);
}
}
@@ -761,7 +754,6 @@ Returns:
UINT8 *FileBuffer;
UINT32 FileSize;
- printf ("PCD tool start.\n");
InputFileName = NULL;
OutputFileName = NULL;
@@ -790,7 +782,5 @@ Returns:
//
WriteOutputFile (OutputFileName);
- printf ("PCD tool done.\n");
-
exit (EXIT_SUCCESS);
}
--
2.8.0.windows.1
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [Patch] BaseTools CommonLib: Remove the unnecessary print message in PcdValueCommon
2018-01-31 10:39 ` Zhu, Yonghong
@ 2018-02-01 8:18 ` Ard Biesheuvel
0 siblings, 0 replies; 3+ messages in thread
From: Ard Biesheuvel @ 2018-02-01 8:18 UTC (permalink / raw)
To: Zhu, Yonghong; +Cc: Gao, Liming, edk2-devel@lists.01.org
On 31 January 2018 at 10:39, Zhu, Yonghong <yonghong.zhu@intel.com> wrote:
> Reviewed-by: Yonghong Zhu <yonghong.zhu@intel.com>
>
This patch breaks the GCC build.
PcdValueCommon.c: In function '__PcdGetPtr':
PcdValueCommon.c:315:11: error: variable 'Byte' set but not used
[-Werror=unused-but-set-variable]
UINT8 Byte;
^~~~
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Liming Gao
> Sent: Wednesday, January 31, 2018 5:04 PM
> To: edk2-devel@lists.01.org
> Subject: [edk2] [Patch] BaseTools CommonLib: Remove the unnecessary print message in PcdValueCommon
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Liming Gao <liming.gao@intel.com>
> ---
> BaseTools/Source/C/Common/PcdValueCommon.c | 10 ----------
> 1 file changed, 10 deletions(-)
>
> diff --git a/BaseTools/Source/C/Common/PcdValueCommon.c b/BaseTools/Source/C/Common/PcdValueCommon.c
> index 92328da..210f87b 100644
> --- a/BaseTools/Source/C/Common/PcdValueCommon.c
> +++ b/BaseTools/Source/C/Common/PcdValueCommon.c
> @@ -330,9 +330,7 @@ Returns:
> break;
> case PcdDataTypePointer:
> Value = &PcdList[Index].Value[1];
> - printf ("Value = %s\n", PcdList[Index].Value);
> for (*Size = 0, Byte = (UINT8) strtoul(Value, &End, 16); Value != End; Byte = (UINT8) strtoul(Value, &End, 16), *Size = *Size + 1) {
> - printf("%x\n", Byte);
> Value = End + 1;
> }
> Buffer = malloc(*Size + 1);
> @@ -401,7 +399,6 @@ Returns:
> PcdList[Index].Value = malloc(Size * 5 + 3);
> PcdList[Index].Value[0] = '{';
> for (ValueIndex = 0; ValueIndex < Size; ValueIndex++) {
> - printf("Value[%d] = %02x\n", ValueIndex, Value[ValueIndex]);
> sprintf(&PcdList[Index].Value[1 + ValueIndex * 5], "0x%02x,", Value[ValueIndex]);
> }
> PcdList[Index].Value[1 + Size * 5 - 1] = '}'; @@ -724,15 +721,11 @@ Returns:
> if (*InputFileName == NULL) {
> fprintf (stderr, "Missing option. Input files is not specified\n");
> exit (EXIT_FAILURE);
> - } else {
> - printf ("Input file name is %s\n", *InputFileName);
> }
>
> if (*OutputFileName == NULL) {
> fprintf (stderr, "Missing option. Output file is not specified\n");
> exit (EXIT_FAILURE);
> - } else {
> - printf ("Output file name is %s\n", *OutputFileName);
> }
> }
>
> @@ -761,7 +754,6 @@ Returns:
> UINT8 *FileBuffer;
> UINT32 FileSize;
>
> - printf ("PCD tool start.\n");
> InputFileName = NULL;
> OutputFileName = NULL;
>
> @@ -790,7 +782,5 @@ Returns:
> //
> WriteOutputFile (OutputFileName);
>
> - printf ("PCD tool done.\n");
> -
> exit (EXIT_SUCCESS);
> }
> --
> 2.8.0.windows.1
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-02-01 8:12 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-01-31 9:03 [Patch] BaseTools CommonLib: Remove the unnecessary print message in PcdValueCommon Liming Gao
2018-01-31 10:39 ` Zhu, Yonghong
2018-02-01 8:18 ` Ard Biesheuvel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox