Re: msadpcm32.acm: Stop converting data instead of crashing

[Bruno Jesus]

On Monday, September 28, 2015, Andrew Eikum  wrote:
> On Sun, Sep 27, 2015 at 12:42:42AM +0800, Bruno Jesus wrote:
> > Signed-off-by: Bruno Jesus <[e-mail] javascript:;>
> >
> > Tomb Raider 3 uses a single archive with all wav files inside, when
> > playing some specific sounds it passes incorrect lengths of data that
> > reach the next wav file header. This causes an assert in the code that
> > is now gracefully handled.
> >
> > Fixes bug https://bugs.winehq.org/show_bug.cgi?id=21000
>
> ...
>
> Looks reasonable, thanks. I think you should change that message to a
> WARN. Audio methods are often called thousands of times per second, so
> spamming an ERR can be very obnoxious.

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.