Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch DirectInput+XInput+XAudio2/OpenAL to use Vortice/SilkNET instead of SlimDX/OpenTK3 #3622

Closed

Conversation

CasualPokePlayer
Copy link
Member

@CasualPokePlayer CasualPokePlayer commented Apr 11, 2023

A lot less painful than SharpAudio was, also just works!

Also see: #2261

EDIT: Also now just replaces DirectInput+XInput alongside the XAudio2/OpenAL stuff.

@CasualPokePlayer CasualPokePlayer changed the title Switch XAudio2SoundOutput to use Vortice instead of SlimDX Switch XAudio2/OpenAL to use Vortice/SilkNET instead of SlimDX/OpenTK3 Apr 11, 2023
This makes SlimDX's Direct3D9 stuff the only usage of SlimDX, a future implementation with Veldrid (or anything else really) will be able to remove SlimDX entirely

(Also of course, SlimDX is used for DirectSound, but that API is ancient/deprecated in favor of XAudio2 and doesn't need to be around when we move to modern .NET. Also of course since we've moved over to newer XAudio2 which is shipped with modern Windows anyways there's probably very little chance of users not having XAudio2)
@CasualPokePlayer CasualPokePlayer changed the title Switch XAudio2/OpenAL to use Vortice/SilkNET instead of SlimDX/OpenTK3 Switch DirectInput+XInput+XAudio2/OpenAL to use Vortice/SilkNET instead of SlimDX/OpenTK3 Apr 12, 2023
also add some debugging code to see what's going on maybe
@CasualPokePlayer
Copy link
Member Author

CasualPokePlayer commented Jun 3, 2023

Seems that Vortice is no longer supports standard 2.0 in its 3.x releases (it only supports net 6+), so we'll have to stay in 2.4.2 for the time being (not really an issue, outside of this slightly annoying hack which won't be needed in 3.x releases)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant