On Monday, September 28, 2015, Andrew Eikum <<a href="mailto:aeikum@codeweavers.com">aeikum@codeweavers.com</a>> 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>
> Signed-off-by: Bruno Jesus <<a href="javascript:;" onclick="_e(event, 'cvml', '00cpxxx@gmail.com')">00cpxxx@gmail.com</a>><br>
><br>
> Tomb Raider 3 uses a single archive with all wav files inside, when<br>
> playing some specific sounds it passes incorrect lengths of data that<br>
> reach the next wav file header. This causes an assert in the code that<br>
> is now gracefully handled.<br>
><br>
> 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>