[Bug 38182] New: King of Dragon Pass: can't change audio volume level
https://bugs.winehq.org/show_bug.cgi?id=38182 Bug ID: 38182 Summary: King of Dragon Pass: can't change audio volume level Product: Wine Version: 1.3.25 Hardware: x86 URL: http://a-sharp.com/files/KoDP-Tour.exe OS: Linux Status: NEW Keywords: download, regression Severity: minor Priority: P2 Component: winmm&mci Assignee: wine-bugs(a)winehq.org Reporter: gyebro69(a)gmail.com CC: aeikum(a)codeweavers.com Regression SHA1: be158e48ad8ee556941bd3f1ff94ca7116680d00 Distribution: --- Adjusting the volume level in the game has no effect, the game is playing audio on the default, highest volume. The game is originally from 1999, I tested with the GOG version. The problem can be reproduced with the demo version too. The same problem in Wine 1.4.1, 1.6.2 and 1.7.38. Last working version: Wine 1.3.25. Regression introduced by commit be158e48ad8ee556941bd3f1ff94ca7116680d00 Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Mon Jul 11 08:28:30 2011 -0500 winmm: Implement waveOut* on top of MMDevAPI. Steps to reproduce the problem in the demo: After installation start the game with tour.exe. Click on the highlighted <Tour> button then click <Skip all> then <Play>. Click on the button right next to <Proceed> in the lower right corner and try to change audio volume level using the slider...you can't. Wine 1.7.38 Fedora 21 Alsa 1.0.28 (Pulseaudio is not running) KoDP-Tour.exe sha1: 445443bd6365f71257abfe0809e9a1932791d1ed -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 --- Comment #1 from Andrew Eikum <aeikum(a)codeweavers.com> --- I can reproduce this. I also tested on a Win7 VM, and the bug does not happen. Probably this is a problem with our winmm mixer implementation, but I didn't dig into it. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 Sagawa <sagawa.aki+winebugs(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sagawa.aki+winebugs(a)gmail.c | |om --- Comment #2 from Sagawa <sagawa.aki+winebugs(a)gmail.com> --- I can confirm the issue, too. This app calls waveOutSetVolume() for volume control. Internally, waveOutSetVolume calls IAudioStreamVolume_SetAllVolumes now. But, currently, ALSA doesn't support in SetAllVolumes [1]. [1] http://source.winehq.org/source/dlls/winealsa.drv/mmdevdrv.c?!v=wine-1.7.38#... . -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 --- Comment #3 from Andrew Eikum <aeikum(a)codeweavers.com> --- Sagawa: Yes, you're right. And on Windows, the in-game slider also affects the volume level in the Windows mixer. We chose not to allow Windows applications to affect the system volume slider, so this is expected behavior. We could do some internal volume control to emulate this, but I really don't want to give Windows applications control over the final, end-user volume levels. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 --- Comment #4 from Béla Gyebrószki <gyebro69(a)gmail.com> --- Still present in 1.8-rc1. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 winetest(a)luukku.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |winetest(a)luukku.com --- Comment #5 from winetest(a)luukku.com --- (In reply to Sagawa from comment #2)
I can confirm the issue, too.
This app calls waveOutSetVolume() for volume control. Internally, waveOutSetVolume calls IAudioStreamVolume_SetAllVolumes now. But, currently, ALSA doesn't support in SetAllVolumes [1].
[1] http://source.winehq.org/source/dlls/winealsa.drv/mmdevdrv.c?!v=wine-1.7. 38#3519 .
"TRACE("ALSA does not support volume control\n");" I am suggesting to change that trace message into FIXME, ERR or even warn? You only see the message when doing tracing. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 joaopa <jeremielapuree(a)yahoo.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jeremielapuree(a)yahoo.fr --- Comment #6 from joaopa <jeremielapuree(a)yahoo.fr> --- Is still a bug in current wine(3.20)? -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 --- Comment #7 from Béla Gyebrószki <gyebro69(a)gmail.com> --- Still valid in wine-3.20-254-g55bcfea8f0. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 Gabriel Ivăncescu <gabrielopcode(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED CC| |gabrielopcode(a)gmail.com Fixed by SHA1| |9d48b3294c0bf73f65ccefae28e | |d225aaba4abab Resolution|--- |FIXED --- Comment #8 from Gabriel Ivăncescu <gabrielopcode(a)gmail.com> --- Fixed by 9d48b3294c0bf73f65ccefae28ed225aaba4abab. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #9 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 4.10. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |4.0.x -- 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.
https://bugs.winehq.org/show_bug.cgi?id=38182 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|4.0.x |--- --- Comment #10 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 4.0.x milestone from bug fixes included in 4.0.3. -- 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.
participants (2)
-
wine-bugs＠winehq.org -
WineHQ Bugzilla