Re: [edk2] [PATCH] MdePkg/BaseLib: Move CHAR_NULL definition to Base.h in BaseLib
Hi Liming, Got it, I will update again. Best Regards, Bell Song > -Original Message- > From: Gao, Liming > Sent: Thursday, November 3, 2016 9:56 AM > To: Song, BinX ; edk2-devel@lists.01.org > Subject: RE: [PATCH] MdePkg/BaseLib: Move CHAR_NULL definition to > Base.h in BaseLib > > Bin: > On CHAR_NULL definition in Base.h, its comment should be Null character. > And, for the second version patch, you need change title to PATCH V2, and > describe what changes in V2. > > > +// > > +// Required unicode control chars ==> Null character > > +// > > +#define CHAR_NULL 0x > > Thanks > Liming > > -Original Message- > > From: Song, BinX > > Sent: Thursday, November 03, 2016 9:31 AM > > To: edk2-devel@lists.01.org > > Cc: Gao, Liming > > Subject: [PATCH] MdePkg/BaseLib: Move CHAR_NULL definition to Base.h > in > > BaseLib > > > > - https://bugzilla.tianocore.org/show_bug.cgi?id=172 > > > > Cc: Liming Gao > > Contributed-under: TianoCore Contribution Agreement 1.0 > > Signed-off-by: Bell Song > > --- > > MdePkg/Include/Base.h | 5 + > > MdePkg/Include/Protocol/SimpleTextIn.h | 1 - > > MdePkg/Library/BaseLib/FilePaths.c | 2 -- > > 3 files changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h > > index 2217058..2441875 100644 > > --- a/MdePkg/Include/Base.h > > +++ b/MdePkg/Include/Base.h > > @@ -338,6 +338,11 @@ struct _LIST_ENTRY { > > /// > > #define NULL ((VOID *) 0) > > > > +// > > +// Required unicode control chars > > +// > > +#define CHAR_NULL 0x > > + > > /// > > /// Maximum values for common UEFI Data Types > > /// > > diff --git a/MdePkg/Include/Protocol/SimpleTextIn.h > > b/MdePkg/Include/Protocol/SimpleTextIn.h > > index 71dcb0b..ebe1c7e 100644 > > --- a/MdePkg/Include/Protocol/SimpleTextIn.h > > +++ b/MdePkg/Include/Protocol/SimpleTextIn.h > > @@ -46,7 +46,6 @@ typedef struct { > > // > > // Required unicode control chars > > // > > -#define CHAR_NULL 0x > > #define CHAR_BACKSPACE0x0008 > > #define CHAR_TAB 0x0009 > > #define CHAR_LINEFEED 0x000A > > diff --git a/MdePkg/Library/BaseLib/FilePaths.c > > b/MdePkg/Library/BaseLib/FilePaths.c > > index c8da6bb..183b323 100644 > > --- a/MdePkg/Library/BaseLib/FilePaths.c > > +++ b/MdePkg/Library/BaseLib/FilePaths.c > > @@ -10,10 +10,8 @@ > >THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" > > BASIS, > >WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER > > EXPRESS OR IMPLIED. > > **/ > > -#include > > #include > > #include > > -#include > > > > /** > >Removes the last directory or file entry in a path by changing the last > > -- > > 2.7.2.windows.1 ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH] MdePkg/BaseLib: Move CHAR_NULL definition to Base.h in BaseLib
Bin: On CHAR_NULL definition in Base.h, its comment should be Null character. And, for the second version patch, you need change title to PATCH V2, and describe what changes in V2. > +// > +// Required unicode control chars ==> Null character > +// > +#define CHAR_NULL 0x Thanks Liming > -Original Message- > From: Song, BinX > Sent: Thursday, November 03, 2016 9:31 AM > To: edk2-devel@lists.01.org > Cc: Gao, Liming > Subject: [PATCH] MdePkg/BaseLib: Move CHAR_NULL definition to Base.h in > BaseLib > > - https://bugzilla.tianocore.org/show_bug.cgi?id=172 > > Cc: Liming Gao > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Bell Song > --- > MdePkg/Include/Base.h | 5 + > MdePkg/Include/Protocol/SimpleTextIn.h | 1 - > MdePkg/Library/BaseLib/FilePaths.c | 2 -- > 3 files changed, 5 insertions(+), 3 deletions(-) > > diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h > index 2217058..2441875 100644 > --- a/MdePkg/Include/Base.h > +++ b/MdePkg/Include/Base.h > @@ -338,6 +338,11 @@ struct _LIST_ENTRY { > /// > #define NULL ((VOID *) 0) > > +// > +// Required unicode control chars > +// > +#define CHAR_NULL 0x > + > /// > /// Maximum values for common UEFI Data Types > /// > diff --git a/MdePkg/Include/Protocol/SimpleTextIn.h > b/MdePkg/Include/Protocol/SimpleTextIn.h > index 71dcb0b..ebe1c7e 100644 > --- a/MdePkg/Include/Protocol/SimpleTextIn.h > +++ b/MdePkg/Include/Protocol/SimpleTextIn.h > @@ -46,7 +46,6 @@ typedef struct { > // > // Required unicode control chars > // > -#define CHAR_NULL 0x > #define CHAR_BACKSPACE0x0008 > #define CHAR_TAB 0x0009 > #define CHAR_LINEFEED 0x000A > diff --git a/MdePkg/Library/BaseLib/FilePaths.c > b/MdePkg/Library/BaseLib/FilePaths.c > index c8da6bb..183b323 100644 > --- a/MdePkg/Library/BaseLib/FilePaths.c > +++ b/MdePkg/Library/BaseLib/FilePaths.c > @@ -10,10 +10,8 @@ >THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" > BASIS, >WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER > EXPRESS OR IMPLIED. > **/ > -#include > #include > #include > -#include > > /** >Removes the last directory or file entry in a path by changing the last > -- > 2.7.2.windows.1 ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
[edk2] [PATCH] MdePkg/BaseLib: Move CHAR_NULL definition to Base.h in BaseLib
- https://bugzilla.tianocore.org/show_bug.cgi?id=172 Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Bell Song --- MdePkg/Include/Base.h | 5 + MdePkg/Include/Protocol/SimpleTextIn.h | 1 - MdePkg/Library/BaseLib/FilePaths.c | 2 -- 3 files changed, 5 insertions(+), 3 deletions(-) diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h index 2217058..2441875 100644 --- a/MdePkg/Include/Base.h +++ b/MdePkg/Include/Base.h @@ -338,6 +338,11 @@ struct _LIST_ENTRY { /// #define NULL ((VOID *) 0) +// +// Required unicode control chars +// +#define CHAR_NULL 0x + /// /// Maximum values for common UEFI Data Types /// diff --git a/MdePkg/Include/Protocol/SimpleTextIn.h b/MdePkg/Include/Protocol/SimpleTextIn.h index 71dcb0b..ebe1c7e 100644 --- a/MdePkg/Include/Protocol/SimpleTextIn.h +++ b/MdePkg/Include/Protocol/SimpleTextIn.h @@ -46,7 +46,6 @@ typedef struct { // // Required unicode control chars // -#define CHAR_NULL 0x #define CHAR_BACKSPACE0x0008 #define CHAR_TAB 0x0009 #define CHAR_LINEFEED 0x000A diff --git a/MdePkg/Library/BaseLib/FilePaths.c b/MdePkg/Library/BaseLib/FilePaths.c index c8da6bb..183b323 100644 --- a/MdePkg/Library/BaseLib/FilePaths.c +++ b/MdePkg/Library/BaseLib/FilePaths.c @@ -10,10 +10,8 @@ THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. **/ -#include #include #include -#include /** Removes the last directory or file entry in a path by changing the last -- 2.7.2.windows.1 ___ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel