<!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,</p>
<p>Thank you for trying to explain the issue, but I must be a little slow today (or there’s something that I am missing).</p>
<p>On 2016-11-04 11:51, Francois Cartegnie wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Le 04/11/2016 à 11:25, Filip Roséen a écrit :</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Fix would hide from tree complete but unknown boxes and bring regressions.</code></pre>
</blockquote>
<pre><code> Thank you for your reply, but I am not sure I completely understand
what you are referring to. Could you elaborate the explanation?</code></pre>
</blockquote>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> From what I can tell the unknown box would simply be discarded if it
could not be parsed, or are you saying that the changes would cause
parsing to stop completely?</code></pre>
</blockquote>
<pre><code> Unknown, non stsd, are -always- present in tree.</code></pre>
</blockquote>
<p>Sure, there might be unknown boxes which we cannot handle - but I still fail to see how the patch would lead to a regression related to how those boxes are handled.</p>
<p>From my understanding other boxes can signal errors in the same way as <code>MP4_ReadBox_default</code> would if the patch is applied, so I must be missing something.</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> | | + udta size 83 offset 600
| | | + creq size 36 offset 608
| | | + cenc size 39 offset 644 (????)</code></pre>
</blockquote>
<p>Could you point me in the direction of a failure related to the patch so that I properly understand why such thing should not be done (not now, nor in the future)?</p>
<p>I am still trying to wrap my head around where the patch could go wrong. I am well aware that you have spent a lot more time with <code>demux/mp4</code> than I have, but I cannot find the point of failure you are talking about.</p>
<p>Best Regards,<br />
Filip</p>
</body>
</html>