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

WASAPI: Improve how default devices are created to support automatic stream routing #754

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

sidit77
Copy link

@sidit77 sidit77 commented Feb 24, 2023

Use ActivateAudioInterfaceAsync to create default input/output devices to support automatic stream routing.

Related issues: #740

Open questions:

  • Fallback for windows versions <1607
  • Implementation of device name/eq
  • What happens when the is no default device (can this even happen?)

…able automatic stream routing in case the default device changes
sidit77 added 2 commits May 2, 2023 20:11
# Conflicts:
#	Cargo.toml
#	src/host/wasapi/device.rs
@snylonue
Copy link

Will the pr be merged? The feature is quite important to handle device changes IMO.

sidit77 added 2 commits May 29, 2024 20:13
# Conflicts:
#	Cargo.toml
#	src/host/wasapi/device.rs
@jacksongoode
Copy link

@est31 I would personally be able to revive/bring this PR forward if you have time to review - no worries if not.

@est31
Copy link
Member

est31 commented Oct 11, 2024

@jacksongoode please, tell me your opinion on it.

@jacksongoode
Copy link

@est31 I guess I'm wondering if there's anything blocking on this or if we could get a review from a maintainer? I'd be happy to make changes with a review from someone more familiar with this library.

@SupernaviX
Copy link

Thank you for working on this PR! I'm switching my app to use it, hopefully it gets upstreamed someday.

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.

5 participants