<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>