[AudioUnit] Adjust framework for a few constants.#25272
[AudioUnit] Adjust framework for a few constants.#25272rolfbjarne wants to merge 1 commit intomainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the AudioUnit bindings so that two notification constants are associated with the correct framework (AudioToolbox), and removes a now-unneeded manual AudioToolbox library handle from the runtime Dlfcn helpers.
Changes:
- Specify
"AudioToolbox"as the framework forkAudioComponentRegistrationsChangedNotificationandkAudioComponentInstanceInvalidationNotification. - Remove the manual
Libraries.AudioToolboxdlopenhandle fromsrc/ObjCRuntime/Dlfcn.cs(theLibrariesclass is otherwise expected to be generated from bindings).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/audiounit.cs | Pins the two notification Field lookups to the AudioToolbox framework to ensure correct constant resolution. |
| src/ObjCRuntime/Dlfcn.cs | Removes the manual AudioToolbox handle from ObjCRuntime.Libraries in Dlfcn.cs. |
✅ [PR Build #23fe724] Build passed (Build packages) ✅Pipeline on Agent |
✅ [PR Build #23fe724] Build passed (Detect API changes) ✅Pipeline on Agent |
✅ 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 #23fe724] Build passed (Build macOS tests) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
🚀 [CI Build #23fe724] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 156 tests passed 🎉 Tests counts✅ cecil: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Monterey (12): All 5 tests passed. [attempt 2] Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
No description provided.