On Monday, September 28, 2015, Andrew Eikum &lt;<a href="mailto:aeikum@codeweavers.com">aeikum@codeweavers.com</a>&gt; wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sun, Sep 27, 2015 at 12:42:42AM +0800, Bruno Jesus wrote:<br>
&gt; Signed-off-by: Bruno Jesus &lt;<a href="javascript:;" onclick="_e(event, &#39;cvml&#39;, &#39;00cpxxx@gmail.com&#39;)">00cpxxx@gmail.com</a>&gt;<br>
&gt;<br>
&gt; Tomb Raider 3 uses a single archive with all wav files inside, when<br>
&gt; playing some specific sounds it passes incorrect lengths of data that<br>
&gt; reach the next wav file header. This causes an assert in the code that<br>
&gt; is now gracefully handled.<br>
&gt;<br>
&gt; Fixes bug <a href="https://bugs.winehq.org/show_bug.cgi?id=21000" target="_blank">https://bugs.winehq.org/show_bug.cgi?id=21000</a><br>...<br>
<br>
Looks reasonable, thanks. I think you should change that message to a<br>
WARN. Audio methods are often called thousands of times per second, so<br>
spamming an ERR can be very obnoxious.<br>
<br></blockquote><div><br></div><div>Will do, thanks for the review. Does this mean I should add a Signed-off-by with your name too? Or that is not related to patch reviewing.</div>