<html><head></head><body>As was pointed out earlier, missing function prototypes always lead to bugs, as parameters are down-converted to int. Now it's even worse because it also breaks CFI.<br><br>But missing initialisers are not always or almost always a bug. Sometimes they are and sometimes, they're not.<br><br><div class="gmail_quote">Le 18 décembre 2019 11:27:57 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-18 10:21, 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;">We've already had this discussion, and this kind of change was already <br>rejected.<br><br>Warnings as errors are screwing anybody using a configuration different <br>from CI's. Packagers have consistently rejected warning as errors.<br></blockquote><br>That's why an option to reject dirty code during development could be <br>useful. I guess we could also add the defines in each target of the CI, <br>it would have the same effect of rejecting dubious code.<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">By your argument, basically all warnings should be errors. You're <br>basically denying the concept of warnings.<br></blockquote><br>You're generalizing my proposal to say I'm against warnings in general.<br><br>We have -Werror-implicit-function-declaration already in configure.ac. <br>Others could be added. It doesn't mean we want to treat all warnings as <br>errors. I certainly don't.<br><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Le 18 décembre 2019 11:16:24 GMT+02:00, Steve Lhomme <robux4@ycbcr.xyz> <br>a écrit :<br><br> On 2019-12-18 10:03, Rémi Denis-Courmont wrote:<br><br> That's exactly what's wrong. They're warnings and they should<br> stay that way.<br><br><br> I disagree.<br><br> You're welcome to attend to them.<br><br><br> I can't compile on macOS, iOS or Android. Should I check the full logs<br> of each MR I'm doing on Gitlab to see if somewhere there's a change I<br> did that is doing something bad ? Will you ?<br><br> Le 18 décembre 2019 10:13:42 GMT+02:00, Steve Lhomme<br> <robux4@ycbcr.xyz><br> a écrit :<br><br> On 2019-12-17 18:20, Rémi Denis-Courmont wrote:<br><br> Le tiistaina 17. joulukuuta 2019, 16.09.28 EET Steve Lhomme a<br> écrit :<br><br> This avoid using pointers of different types. It may lead to<br> serious issues<br> in the code so rather than a warning, it should be<br> avoided/fixed.<br><br> This will raise warnings in harmless cases of implicit<br> conversions. Some of<br><br><br> It's not adding any warning. It's turning existing (unattended)<br> warnings<br> into errors.<br><br> those might not be easily fixable - coming from external headers.<br><br><br> True, but it seems we have none of these on Windows, Linux (and<br> macOS<br> I'm told). I'd rather not allow new code using dirty pointer<br> casts, even<br> from external library headers.<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><br><br><br> -- <br> Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez<br> excuser<br> ma brièveté.<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><br><br><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser <br>ma brièveté.<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><br><br></blockquote><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>