[ObjCRuntime] Enable nullability and clean up BackingField.#24734
[ObjCRuntime] Enable nullability and clean up BackingField.#24734rolfbjarne merged 3 commits intomainfrom
Conversation
This is file 4 of 7 files with nullability disabled in ObjCRuntime. * Enable nullability (#nullable enable). * Add nullable annotations for cached backing-field helper APIs. * Add converter null checks using ThrowHelper to improve static analysis. * Add XML documentation for BackingField and its public helper methods. Contributes towards #17285. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Enables nullable reference types in ObjCRuntime’s BackingField helper to improve static analysis and better document the backing-field caching pattern used by bindings.
Changes:
- Enabled
#nullable enableforBackingField.cs. - Updated
BackingField.Get/Savesignatures to use nullable backing fields (T?) and added null-check for the converter. - Added XML documentation for
BackingFieldand its public helper methods.
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.
…nullability-backingfield-04
✅ [CI Build #9c3ebe1] Build passed (Build packages) ✅Pipeline on Agent |
✅ [PR Build #9c3ebe1] 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 #9c3ebe1] Build passed (Build macOS tests) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
🚀 [CI Build #9c3ebe1] 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. Html Report (VSDrops) Download Pipeline on Agent |
This is file 4 of 7 files with nullability disabled in ObjCRuntime.
Contributes towards #17285.