Re: [Patch] MdePkg Base.h: Update field name in VERIFY_UINTXX_ENUM_SIZE to follow style

["Kinney, Michael D" <michael.d.kinney@intel.com>]

Liming,

I agree enums should start with [A-Z].

These enums are only use to verify compiler config is right
and I wanted to make sure the enum names did not conflict
with any other names that might be used.  This is why
I added '__' to the beginning of these enum names.  These
enums would never be used my any code after the verify 
operation in the same .h file.

I don't expect name collisions without the '__', so I am
ok with this patch.

But, we could also consider adding a comment that the '__'
is on purpose.

Mike

> -----Original Message-----
> From: Gao, Liming
> Sent: Thursday, November 10, 2016 7:33 PM
> To: edk2-devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: [Patch] MdePkg Base.h: Update field name in VERIFY_UINTXX_ENUM_SIZE to
> follow style
> 
> For field name in structure, its first character should be upper case.
> 
> Cc: Michael Kinney <michael.d.kinney@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Liming Gao <liming.gao@intel.com>
> ---
>  MdePkg/Include/Base.h | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h
> index 5e24b5d..749c275 100644
> --- a/MdePkg/Include/Base.h
> +++ b/MdePkg/Include/Base.h
> @@ -64,15 +64,15 @@ VERIFY_SIZE_OF (CHAR8, 1);
>  VERIFY_SIZE_OF (CHAR16, 2);
> 
>  typedef enum {
> -  __VerifyUint8EnumValue = 0xff
> +  VerifyUint8EnumValue = 0xff
>  } __VERIFY_UINT8_ENUM_SIZE;
> 
>  typedef enum {
> -  __VerifyUint16EnumValue = 0xffff
> +  VerifyUint16EnumValue = 0xffff
>  } __VERIFY_UINT16_ENUM_SIZE;
> 
>  typedef enum {
> -  __VerifyUint32EnumValue = 0xffffffff
> +  VerifyUint32EnumValue = 0xffffffff
>  } __VERIFY_UINT32_ENUM_SIZE;
> 
>  VERIFY_SIZE_OF (__VERIFY_UINT8_ENUM_SIZE, 4);
> --
> 2.8.0.windows.1