<p>Hmm, I don’t have that line of code in the codebase anymore - I will however resubmit this patch-batch due to major cleanup that have changed things (and as such the patch no longer apply).</p>
<p>Though I tested the patches (in this thread) on a fresh git pull before I sent it to <code>vlc-devel</code>, but the dangerous c-style cast is probably to blame for not resulting in a compile error.</p>
<p>I have replaced as many c-style casts as I can find in earlier patches, before I got bitten by a really nasty bug where a variable was name <code>ctime</code> prior to the modification, and I changed it to another name (forgetting to change one instance of where <code>ctime</code> was used).</p>
<p>The problem? A c-style cast such as <code>uint32( ctime )</code> was used later, and everything compiled fine because <code>time.h</code> is implicitly included. The compiler then happily converted the <em>function pointer</em> to an <code>uint32</code>.</p>
<p>The new patch-block should fix such hard-to-track bugs since the appropriate c++ casts have replaced the c-style ones.</p>
<p>Thanks!</p>
<hr style="height:1px;margin-bottom:20px;background-color:#ddd;color:#ddd" />
<p>On 16/03/04 11:16, Francois Cartegnie wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>Le 03/03/2016 13:30, Filip Roséen a écrit :</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<pre><code>  E_CASE( KaxAudioChannels, achan ) {
    vars.p_tk->fmt.audio.i_channels = uint8( chan );
  }</code></pre>
</blockquote>
<p>I’m not sure what is the issue with KaxParsing, but I doubt this patch compiles with the above typo :p</p>
<p>Francois</p>
</blockquote>