<div dir="ltr"><br><br><div class="gmail_quote">On Wed, Aug 13, 2008 at 6:28 PM, Pierre d'Herbemont <span dir="ltr"><<a href="mailto:pdherbemont@free.fr">pdherbemont@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="Ih2E3d"><br>
On Aug 13, 2008, at 4:19 PM, git version control wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
vlc | branch: master | Rémi Denis-Courmont <<a href="mailto:rdenis@simphalempin.com" target="_blank">rdenis@simphalempin.com</a>> | Wed Aug 13 17:21:54 2008 +0300| [481f1d5862c70e9e5f2b076256a5de63b02b33a9] | committer: Rémi Denis-Courmont<br>

<br>
Revert "Turn on -Werror for libvlccore and libvlc."<br>
<br>
Do not do this with portable code. Ever.<br>
</blockquote>
<br></div>
IMO that's a shame. We shouldn't have all that valid warnings (asprintf to name it).<br>
<br>
The portable code argument isn't valuable as we do have build bots.</blockquote><div><br></div><div>Before any comment, it's true that this could be annoying and that most of us may get a broken a tree most of the time, if no attention is paid.</div>
<div><br></div><div>But I still think we should give it a try at least to get rid of most dangerous warnings.</div><div><br></div><div>Pierre.</div></div></div>