Project

General

Profile

Actions

Emulator Issues #4601

closed

Audio backend cannot be changed

Added by hatarumoroboshi almost 13 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
No
Relates to usability:
No
Relates to performance:
No
Easy:
No
Relates to maintainability:
No
Regression start:
Fixed in:

Description

What's the problem?
XAudio2 plugin selection doesn't work (it cannot be saved anymore).

Dolphin version with the problem (as it appears in the title bar, Ex.: "R
4779" or "R 6403M"):
r7577 and later

(optional) Dolphin version that does not have the problem:
r7576

Operating system and version:
32-bit or 64-bit:
WINDOWS XP 32bit

Game ID (as it appears in game properties, Ex.: "GZ2P01" or "RSBE01"):

Build command-line (not on Windows):

Was the ISO a plain dump from disc, compressed and/or scrubbed?

Please provide any additional information below.
If you select XAudio2 in the DSP tab and then exit (save), the option doesn't get saved because when you re-enter in the DSP tab DSound is still selected.


Related issues 1 (0 open1 closed)

Blocks Emulator - Emulator Issues #4270: Dolphin 3.0 bug trackerFixedNeoBrainX

Actions
Actions #1

Updated by BhaaL almost 13 years ago

Does commenting out line 918 of ConfigMain.cpp (should read "break;" in front of "default:") fix this?

Actions #2

Updated by MofoMan2000 almost 13 years ago

Lol yeah I'm stuck with XAudio2.

This issue should be renamed to "Audio backend cannot be changed"

Actions #3

Updated by BhaaL almost 13 years ago

Might be a good idea to fix this before 3.0 I'd say :)

I'll see if I can compile dolphin on this box, maybe I'll look into it right away...

Actions #5

Updated by mickamartel almost 13 years ago

it works, but you must switch the audio config and close the box and re-open the box and make your choice to select xaudio & 48KHz -_-" (it's not an huge problem but it's irritating)

Actions #6

Updated by BhaaL almost 13 years ago

I too noticed that the code looks a bit odd. That default case seems to handle pretty much everything, all at once. I don't know why it was coded like that, but it might be time to invest a bit more time there than just to remove the break.

Can't build or commit on this box tho =/

Actions #7

Updated by NeoBrainX almost 13 years ago

  • Status changed from New to Fixed

This issue was closed by revision r7605.

Actions

Also available in: Atom PDF