<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div><br></div><div>On Fri, Feb 28, 2020, at 07:34, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" id="qt"><div>This patch enables people to use outdated compilers that will miss modern optimisations and threat mitigation features from newer LLVM versions, which are available for all platforms that any VLC dev cares about. All of that to avoid one apt-get install or equivalent.<br></div><div><br></div><div>15 years ago, when GCC was king and slow moving, supporting old compilers didn't make much difference. Now it does.<br></div><div><br></div><div>I completely disagree.<br></div></blockquote><div><br></div><div>That is a valid point, I propose that we continue this discussion tomorrow.<br></div><div><br></div><blockquote type="cite" id="qt"><div><br></div><div class="qt-gmail_quote"><div>Le 27 février 2020 17:06:56 GMT+02:00, Romain Vimont <rom1v@videolabs.io> a écrit :<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:0pt;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><pre class="qt-k9mail"><div><br></div><div><br></div><div>On 2/27/20 3:57 PM, Thomas Guillem wrote:<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:1ex;margin-left:0.8ex;border-left-color:rgb(114, 159, 207);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><div>On Thu, Feb 27, 2020, at 15:50, Rémi Denis-Courmont wrote:<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:1ex;margin-left:0.8ex;border-left-color:rgb(173, 127, 168);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><div>Point being, it's a compiler bug, which is already fixed. Don't use <br></div><div>old buggy EOL compilers. This patch is wrong.<br></div></blockquote><div>But we are still stating that VLC 4.0 can be built with "buggy EOL <br></div><div>compilers" like you said.<br></div><div><br></div><div>So, the issue can't be closed until we either:<br></div><div>  - Bump the compiler version from NEWS<br></div><div>  - Apply this patch (or do any other fixes).<br></div></blockquote><div><br></div><div>This patch just adds an harmless cast.<br></div><div><br></div><div>Whatever the decisions about which compiler we support or not support, <br></div><div>if this tiny change allows to support more compilers, I'd prefer it to <br></div><div>be merged.<br></div><div><br></div><div>Just my 2 cts.<hr>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></div></pre></blockquote></div><div><br></div><div>-- <br></div><div>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté. <br></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div></blockquote><div><br></div></body></html>