<p>Hey Denis,</p>
<p>On 16/08/05 17:11, Denis Charmet wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> On 2016-08-05 15:01, Filip Roséen wrote:</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code>I agree on all points raised above, which leads to follow-up questions:

    * would it be appropriate to completely deprecate the macro (and
replace usage of such), or;
    * should we fix it in a way that could make the intention more clear, or;
    * let it be and intentionally look the other way when such is used/not used?</code></pre>
</blockquote>
<pre><code> Unless you feel like removing it everywhere that's exactly what's going to happen.</code></pre>
</blockquote>
<p>Which is why I asked for the most suitable approach to the matter, just as I asked about the current inconsistency regarding <code>pf_readdir</code> and <code>STREAM_IS_DIRECTORY</code> in the following discussion (which has not yet received any replies):</p>
<ul>
<li>https://mailman.videolan.org/pipermail/vlc-devel/2016-July/108687.html</li>
</ul>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> There are always more urgent matters than code janitorial. It's the
 principle of volunteer code, you only scratch your own nose. :)</code></pre>
</blockquote>
<p>I was thinking that such janitorial issues could be added to the somewhat obsolete list of such tasks under the following wiki-pages:</p>
<ul>
<li>https://wiki.videolan.org/Mini_Projects/</li>
<li>https://wiki.videolan.org/GCodeIn_Ideas/</li>
</ul>
<p>Even if matters such as this are not exactly <em>“urgent”</em>, it certainly helps knowing the community praxis when writing new code.</p>
<p>Having to consider <em>not using-</em> vs <em>using a certain helper</em> when implementing stuff is a minor obstacle that I would rather not be there at all.</p>