-
Notifications
You must be signed in to change notification settings - Fork 124
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
OpenSL ES full duplex restored #196
OpenSL ES full duplex restored #196
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't reviewed 62998e5 on the basis that it was reviewed for the last landing and hasn't been changed since (i.e. it's a pure revert of the backout).
Everything else looks good.
@@ -1360,6 +1361,9 @@ opensl_stream_init(cubeb * ctx, cubeb_stream ** stream, char const * stream_name | |||
assert(r == 0); | |||
|
|||
if (output_stream_params) { | |||
LOG("Playback params: Rate %d, channels %d, format %d, latency in frames %d.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rate, channels, and latency_frames are unsigned so should be %u. LOG() should be producing -Wformat warnings for these now (on GCC and Clang), please let me know if you're not seeing them.
@@ -1368,6 +1372,9 @@ opensl_stream_init(cubeb * ctx, cubeb_stream ** stream, char const * stream_name | |||
} | |||
|
|||
if (input_stream_params) { | |||
LOG("Capture params: Rate %d, channels %d, format %d, latency in frames %d.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto unsigned
Full duplex functionality for OpenSL ES. The patch in the bottom is the same patch which backout in 99d6ae1 rebased to current version. The rest of the patches are all problems found during build, manual testing and try run. Latest try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=cab4bbde79b65b872f0181b2076d49b3b8df231d
Latest autophone try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=ae10057481627d78a7b47483adfc27768c985d79
Review @kinetiknz and/or @padenot ?