<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<meta http-equiv="Content-Style-Type" content="text/css" />
<meta name="generator" content="pandoc" />
<title></title>
<style type="text/css">code{white-space: pre;}</style>
</head>
<body>
<p>Hi Rémi,</p>
<p>On 2017-02-26 21:55, Rémi Denis-Courmont wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Le sunnuntaina 26. helmikuuta 2017, 20.50.57 EET Filip Roséen a écrit :</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Hi Rémi,
On 2017-02-26 21:28, Rémi Denis-Courmont wrote:</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Nack.</code></pre>
</blockquote>
<pre><code> Care to elaborate the above statement; exactly what is it that you are
*"nacking"*?</code></pre>
</blockquote>
<pre><code> I am nacking the entire patch. It is pure luck if it does not break the test
suite (I have not checked).</code></pre>
</blockquote>
<p>As already stated, given that you nacked an obvious dependency it is quite clear that you are not a fan of what follows.</p>
<p>Regarding the test suite I do not see how the patches in question changes any behavior other than not making use of attributes when the compiler either rejects or will ignore them completely (at least that is the underlying idea).</p>
<p>I am sorry if I missed something that really does change the behavior, other than what was previously stated by you in the message linked below:</p>
<ul>
<li>https://mailman.videolan.org/pipermail/vlc-devel/2017-February/111752.html</li>
</ul>
</body>
</html>