[Bug 33055] Fishdom 2 crashes during the initial loading stage

[wine-bugs at winehq.org]

http://bugs.winehq.org/show_bug.cgi?id=33055

Rico <kgbricola at web.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |aeikum at codeweavers.com

--- Comment #7 from Rico <kgbricola at web.de> 2013-04-07 09:49:23 CDT ---
Removing the assert and using only the warning
(http://bugs.winehq.org/attachment.cgi?id=44088) or reverting to the old
behavior will work till patch:

2c6087457b97e815ee094e7ed75ca79317a5d14a is the first bad commit
commit 2c6087457b97e815ee094e7ed75ca79317a5d14a
Author: Andrew Eikum <aeikum at codeweavers.com>
Date:   Mon May 14 12:31:01 2012 -0500

    dsound: Remove minlen, since we always mix full chunks anyway.

:040000 040000 52dc0b21b6cd59b8407f39442fb156e3d6112091
32eb1f192f1ec78aafee2cec32e739e78f874a17 M    dlls

After that using the dirty workaround is needed!

So 2c6087457b97e815ee094e7ed75ca79317a5d14a introduces something which really
breaks - regardless of the regression by
83761d20a8654a616b557ecdb2869436beae7f95. Maybe the app puts also some garbage
values in there? Andrew could you have a look please?

I think using the dirty workaround is still the way to go, but we should take a
look not introducing some other problem with
2c6087457b97e815ee094e7ed75ca79317a5d14a.

-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.