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
[osx] utilize planar support from AE in the OSX sink #4774
Conversation
|
||
/* For planar audio we set m_frameSize to be the size per plane */ | ||
if (m_planes > 1) | ||
m_format.m_frameSize /= m_planes; |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Updated. Also included some fixes for bitstreaming (which is likely not going to be planar, but at least it now correctly takes things from the planes if it is) and a bit of cleanup. @Memphiz mind reviewing? jenkins build this please |
Retested (slight changes compared to the "previous" implementation) against kXAudioDriver, still gives flawless audio output for all channel setups and all input sample rates. Can't test S/PDIF passthrough though. AppleHDA 2-channel-interleaved remains intact. |
looks good to me :) |
Thanks! Will rebase down and build test. |
…ultiple small memcpy's and it's not as if we're performance constrained here
… rather than m_format.m_sampleRate
jenkins build this please |
[osx] utilize planar support from AE in the OSX sink
@Memphiz @FernetMenta please take a look. Thanks!