Re: [edk2-devel] [PATCH v3 1/1] MdePkg: Add STATIC_ASSERT macro

["Philippe Mathieu-Daudé" <philmd@redhat.com>]

On 8/17/19 1:28 AM, Vitaly Cheptsov via Groups.Io wrote:
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2048
> 
> Provide a macro for compile time assertions.
> Equivalent to C11 static_assert macro from assert.h.
> 
> Signed-off-by: Vitaly Cheptsov <vit9696@protonmail.com>
> ---
>  MdePkg/Include/Base.h | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h
> index ce20b5f01d..ec096133ba 100644
> --- a/MdePkg/Include/Base.h
> +++ b/MdePkg/Include/Base.h
> @@ -843,6 +843,20 @@ typedef UINTN  *BASE_LIST;
>  #define OFFSET_OF(TYPE, Field) ((UINTN) &(((TYPE *)0)->Field))
>  #endif
>  
> +/**
> +  Portable definition for compile time assertions.
> +  Equivalent to C11 static_assert macro from assert.h.
> +
> +  @param  Expression  Boolean expression.
> +  @param  Message     Raised compiler diagnostic message when expression is false.
> +
> +**/
> +#ifdef _MSC_EXTENSIONS
> +  #define STATIC_ASSERT static_assert
> +#else
> +  #define STATIC_ASSERT _Static_assert
> +#endif
> +
>  /**
>    Macro that returns a pointer to the data structure that contains a specified field of
>    that data structure.  This is a lightweight method to hide information by placing a
> 

Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>