<html><head></head><body>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><br>15 years ago, when GCC was king and slow moving, supporting old compilers didn't make much difference. Now it does.<br><br>I completely disagree.<br><br><div class="gmail_quote">Le 27 février 2020 17:06:56 GMT+02:00, Romain Vimont <rom1v@videolabs.io> 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"><br><br>On 2/27/20 3:57 PM, Thomas Guillem wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">On Thu, Feb 27, 2020, at 15:50, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">Point being, it's a compiler bug, which is already fixed. Don't use <br>old buggy EOL compilers. This patch is wrong.<br></blockquote>But we are still stating that VLC 4.0 can be built with "buggy EOL <br>compilers" like you said.<br><br>So, the issue can't be closed until we either:<br>  - Bump the compiler version from NEWS<br>  - Apply this patch (or do any other fixes).<br></blockquote><br>This patch just adds an harmless cast.<br><br>Whatever the decisions about which compiler we support or not support, <br>if this tiny change allows to support more compilers, I'd prefer it to <br>be merged.<br><br>Just my 2 cts.<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>