
<!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 Steve,</p>

<p>On 2016-11-02 09:16, Steve Lhomme wrote:</p>

<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">

<pre><code> The description of the fix seems logical but that's not what the code

 does. I think EnsureDuration should keep the duration set to -1 if the

 list of clusters is empty, it should not crash.</code></pre>

</blockquote>

<p>Of course; EnsureDuration should not crash, but it currently assumes that at least one cluster is present in the file, which causes it to crash (on broken files where there is no cluster).</p>

<p>Given that a cluster is mandatory, there is no need to call the function at all if there is no cluster; we won’t be able to play the file in either case.</p>

<ul>

<li>In what way does the patch not do what the description states it does?</li>

</ul>

</body>

</html>