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

Darwin fixes #390

Merged
merged 2 commits into from
Aug 27, 2024
Merged

Darwin fixes #390

merged 2 commits into from
Aug 27, 2024

Conversation

james-pellow
Copy link
Contributor

fix: Exception where format.sampleRate != hwFormat.sampleRate
This patch fixes an exception that crashes the app. The exception is caused by a wrong source audio format being used for the installTap call on the input. Instead of the inputFormat being used, the outputFormat was used which caused the exception when some devices are selected.

feat(macOS): Support device by deviceID as well as deviceUID
For compatibility with other packages which may report devices by their deviceID rather than deviceUID. The flutter_webrtc package is an example. This patch will match on either. There is no overlap so the old behavior should not be impacted.

@llfbandit llfbandit merged commit 97f4032 into llfbandit:master Aug 27, 2024
@llfbandit
Copy link
Owner

Thanks for your contribution!

@james-pellow james-pellow deleted the darwin-fixes branch August 27, 2024 19:18
@james-pellow
Copy link
Contributor Author

Thanks! I didn't change the version in pubspec.yaml to 1.1.3 as I wasn't sure if you were ready to release yet. We look forward to having the changes in a published version. Is that something you are ready to do?

@llfbandit
Copy link
Owner

I may tackle some more issues on darwin package before publishing.
But I can publish it already (after some testing) before more fixing which will likely imply refactoring and some delay.
Let me know.

@james-pellow
Copy link
Contributor Author

That sounds good. I just have a dependency_overrides entry for now, so its not a big fire but I like to keep those overrides to a minimum.

@llfbandit
Copy link
Owner

I agree, if other fixes take me too far I will publish.

@llfbandit
Copy link
Owner

Released in 1.1.2

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.

2 participants