[vlc-devel] [PATCH 3/3] configure: add -Werror=missing-field-initializers option to the compiler

Rémi Denis-Courmont remi at remlab.net
Tue Dec 17 18:22:41 CET 2019


Le tiistaina 17. joulukuuta 2019, 16.09.29 EET Steve Lhomme a écrit :
> This avoids leaving some fields initialized in structures that require
> initialization (like in the video context operations). It's always better to
> set NULL (with a TODO/FIXME is needed) explicitly than forget it.
> 
> Compound literal designated initializers are not affected.

That's great but designated initializers don't work in C++, or in common C/C++ 
headers, and more generally, perfectly valid code that just wants to 
initialize values to the default nul.

This does not belong as an error. Denis and I have already made the exact same 
point not that long ago.

-- 
雷米‧德尼-库尔蒙
http://www.remlab.net/





More information about the vlc-devel mailing list