<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 22, 2015 at 10:40 AM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 2015-06-22 17:31, Hugo Beauzée-Luyssen a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 22/06/2015 16:15, Rémi Denis-Courmont wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
This is pointless if not actively harmful. Our public headers must be<br>
usable in external C++ applications code with or without the std<br>
namespace. AFAIU, this change hides bug that will then strike third<br>
parties after we release.<br>
<br>
</blockquote>
<br>
My understanding of public headers in this context is "anything in<br>
the include/vlc folder", which isn't affected by this commit.<br>
</blockquote>
<br>
Public headers is anything that is copied by make install. That is:<br>
- for external apps, the LibVLC API headers in include/vlc/ and,<br>
- for external plugins, most but not all of VLC plugin API headers in include/.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm not sure to understand what you mean here.<br>
</blockquote>
<br>
If we need to depend on a library that has "using namespace std;" in its public header, then what?<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Not that I disagree with earlier points, but said library would be acting crazy. Putting "using namespace std" in a header is extremely bad form, especially in a library's public header.<br></div></div><br></div><div class="gmail_extra">Best,<br></div><div class="gmail_extra">Tristan<br></div></div>