-
Notifications
You must be signed in to change notification settings - Fork 511
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
[appkit] Remove leftover code made obsolete by newrefcount. #15103
Conversation
Remove code that we needed at some point due to memory related issues. These issues were fixed several years ago (newrefcount), so we no longer need this code (in fact it causes problems, see xamarin#15089).
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.
src/appkit.cs
Outdated
@@ -15929,16 +15929,12 @@ partial interface NSView : NSDraggingDestination, NSAnimatablePropertyContainer, | |||
void WillRemoveSubview ([NullAllowed] NSView subview); | |||
|
|||
[Export ("removeFromSuperview")] | |||
[PreSnippet ("var mySuper = Superview;", Optimizable = true)] | |||
[PostSnippet ("if (mySuper != null) {\n\t#pragma warning disable 168\n\tvar flush = mySuper.Subviews;\n#pragma warning restore 168\n\t}", Optimizable = true)] | |||
void RemoveFromSuperview (); | |||
|
|||
[Export ("replaceSubview:with:")][PostGet ("Subviews")] |
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.
the [PostGet]
, like this one (there are other in the type), are also leftovers from that era and could be removed (but they are not causing the same problem).
This comment has been minimized.
This comment has been minimized.
💻 [PR Build] Tests on macOS Mac Catalina (10.15) passed 💻✅ All tests on macOS Mac Catalina (10.15) passed. Pipeline on Agent |
❌ [PR Build] Tests on macOS M1 - Mac Big Sur (11.5) failed ❌Failed tests are:
Pipeline on Agent |
📋 [PR Build] API Diff 📋API diff (for current PR)ℹ️ API Diff (from PR only) (please review changes) .NETXamarin vs .NETAPI diff (vs stable)✅ API Diff from stable .NETXamarin vs .NETGenerator diffℹ️ Generator Diff (please review changes) Pipeline on Agent XAMBOT-1044.Monterey' |
Remove code that we needed at some point due to memory related issues. These
issues were fixed several years ago (newrefcount), so we no longer need this
code (in fact it causes problems, see #15089).