-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Ownership tracking #89329
Ownership tracking #89329
Conversation
…lingInterfaceCleanup
…lingInterfaceCleanup
…lingInterfaceCleanup
…lingInterfaceCleanup
…lingInterfaceCleanup
…lingInterfaceCleanup
…ld 20230716.1 (dotnet#89077)" This reverts commit 09e724f.
…slyn build 20230716.1 (dotnet#89077)"" This reverts commit 0a532ad.
…lingInterfaceCleanup
…lingInterfaceCleanup
Tagging subscribers to this area: @dotnet/interop-contrib Issue DetailsBuilds off of #89139. Implements AssignOut in unmanaged to managed stubs for parameters that need to be marshaled back to the native caller. The values are marshalled into a local variable, then assigned to the parameter at the end of the method. The local variable is cleaned up in the catch clause before returning, and the parameter value is cleaned up after the try catch before returning successfully. The code isn't cleaned up, but the diff at https://github.com/jtschuster/GeneratedCode/pull/2/files can help show what the changes do compared to what's in main.
|
This is going to look a lot different with the new changes coming in, closing this |
Builds off of #89139.
Implements AssignOut in unmanaged to managed stubs for parameters that need to be marshaled back to the native caller. The values are marshalled into a local variable, then assigned to the parameter at the end of the method. The local variable is cleaned up in the catch clause before returning, and the parameter value is cleaned up after the try catch before returning successfully.
The code isn't cleaned up, but the diff at https://github.com/jtschuster/GeneratedCode/pull/2/files can help show what the changes do compared to what's in main.