[AudioToolbox] Improve AudioQueue API a little bit. Fixes #11183.#23988
[AudioToolbox] Improve AudioQueue API a little bit. Fixes #11183.#23988rolfbjarne merged 3 commits intomainfrom
Conversation
In the issue #11183, it becomes the API to allocate and use audio queue buffers is less than ideal, because the developer has to cast IntPtrs to AudioQueueBuffer pointers. This is not obvious at all, so create a better API where we return AudioQueueBuffer pointers directly. Also improve the API in the AudioQueueBuffer struct to support spans + improve xml docs as well. And finally add more tests. Fixes #11183.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✅ [PR Build #b3d9956] Build passed (Detect API changes) ✅Pipeline on Agent |
✅ [CI Build #b3d9956] Build passed (Build packages) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
🔥 [CI Build #b3d9956] Test results 🔥Test results❌ Tests failed on VSTS: test results 0 tests crashed, 50 tests failed, 63 tests passed. Failures❌ dotnettests tests (iOS)Details
Html Report (VSDrops) Download ❌ dotnettests tests (MacCatalyst)Details
Html Report (VSDrops) Download ❌ dotnettests tests (macOS)Details
Html Report (VSDrops) Download ❌ dotnettests tests (tvOS)Details
Html Report (VSDrops) Download ❌ linker testsDetails
|
✅ [CI Build #b3d9956] Build passed (Build macOS tests) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
💻 [CI Build #b3d9956] Tests on macOS X64 - Mac Sonoma (14) passed 💻✅ All tests on macOS X64 - Mac Sonoma (14) passed. Pipeline on Agent |
💻 [CI Build #b3d9956] Tests on macOS M1 - Mac Ventura (13) passed 💻✅ All tests on macOS M1 - Mac Ventura (13) passed. Pipeline on Agent |
💻 [CI Build #b3d9956] Tests on macOS arm64 - Mac Tahoe (26) passed 💻✅ All tests on macOS arm64 - Mac Tahoe (26) passed. Pipeline on Agent |
💻 [CI Build #b3d9956] Tests on macOS arm64 - Mac Sequoia (15) passed 💻✅ All tests on macOS arm64 - Mac Sequoia (15) passed. Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
💻 [CI Build #b3d9956] Tests on macOS M1 - Mac Monterey (12) passed 💻✅ All tests on macOS M1 - Mac Monterey (12) passed. Pipeline on Agent |
🚀 [CI Build #b3d9956] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 118 tests passed 🎉 Tests counts✅ cecil: All 1 tests passed. Html Report (VSDrops) Download Pipeline on Agent |
In the issue #11183, it becomes the API to allocate and use audio queue buffers is
less than ideal, because the developer has to cast IntPtrs to AudioQueueBuffer pointers.
This is not obvious at all, so create a better API where we return AudioQueueBuffer
pointers directly.
Also improve the API in the AudioQueueBuffer struct to support spans + improve xml docs as well.
And finally add more tests.
Fixes #11183.