Conversation
- Fix Delegate.Method and Delegate.Target for marshalled delegates - Add tests and fixes for corner various delegate corner case behaviors - Delete runtime test for GetInvocationList since there is a better test coverage for this API under libraries Fixes dotnet/runtimelab#164
|
Tagging subscribers to this area: @dotnet/area-meta Issue Details
Fixes dotnet/runtimelab#164
|
|
Tagging subscribers to this area: @agocke, @MichalStrehovsky, @jkotas Issue Details
Fixes dotnet/runtimelab#164
|
|
While reviewing I noticed the https://gist.github.com/AustinWise/87eb03c4ec0cd06564c79e9b33bcfc07 One possible fix: make GCHandle in PInvokeDelegateThunk be WeakTrackResurrection. In the PInvokeDelegateThunk finalizer check to see if the object pointed to by the handle is still alive. If so, call ReRegisterForFinalize to defer finalization. |
99a99e9 to
7a2df9b
Compare
|
/azp run runtime-nativeaot-outerloop |
|
Azure Pipelines successfully started running 1 pipeline(s). |
@AustinWise Thank you! I have included the fix and the test in this PR. |
|
/azp run runtime-nativeaot-outerloop |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
This is ready for review |
…into DelegateMethodInfo
|
/azp run runtime-nativeaot-outerloop |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Fixes dotnet/runtimelab#164