
<html><head></head><body>You can't always use explicit initialisation. Some structures have partially system-dependent layout.<br><br><div class="gmail_quote">Le 18 décembre 2019 10:19:17 GMT+02:00, Steve Lhomme <robux4@ycbcr.xyz> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<pre class="k9mail">On 2019-12-17 18:22, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Le tiistaina 17. joulukuuta 2019, 16.09.29 EET Steve Lhomme a écrit :<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"> This avoids leaving some fields initialized in structures that require<br> initialization (like in the video context operations). It's always better to<br> set NULL (with a TODO/FIXME is needed) explicitly than forget it.<br><br> Compound literal designated initializers are not affected.<br></blockquote>That's great but designated initializers don't work in C++, or in common C/C++<br></blockquote><br>Then don't use such initializers in C++ code.<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">headers, and more generally, perfectly valid code that just wants to<br>initialize values to the default nul.<br></blockquote><br>It's better to explicitly set NULL for a callback IMO than not paying <br>attention at all in code you don't know about. That will only be found <br>in the bots if we can detect such missed code change early.<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">This does not belong as an error. Denis and I have already made the exact same<br>point not that long ago.<br></blockquote><br>How about an option to force extra compilation checks ?<hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>