Re: [edk2-devel] [PATCH] Use C99 flexible arrays

["Rebecca Cran" <rebecca@bsdio.com>]

I don't think I got around to explicitly requiring C11 with -std=c11, 
but since we use _Static_assert on compilers that don't have MSC 
Extensions or are C++ compilers we implicitly require it. It's possible 
it's implemented via an extension though.


-- 

Rebecca Cran


On 8/23/23 18:36, Pedro Falcato wrote:
> On Wed, Aug 23, 2023 at 10:41 PM Rebecca Cran <rebecca@bsdio.com> wrote:
>> With Visual Studio 2019 adding C11 and C17 support as first-class
>> features, I think we should be safe nowadays to rely on them. But we
>> should check if older versions that we still support (VS 2015 etc.)
>> support them too.
>>
>> https://devblogs.microsoft.com/cppblog/c11-and-c17-standard-support-arriving-in-msvc/
> Don't we explicitly rely on C11 now? Due to your __function__ changes?
>
> Anyway, I would recommend Elyes to break this change up into a patch
> set with per-package patches. This way maintainers could easily review
> and test the changes. The way it is right now isn't very reviewable
> nor testable.
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#108023): https://edk2.groups.io/g/devel/message/108023
Mute This Topic: https://groups.io/mt/100861513/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-