[release/8.0] Update Marshal.QueryInterface() argument modifier
#91984
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Change "in" modifier to "ref readonly" to avoid warnings in existing interop code.
|
Note regarding the This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change. |
|
Tagging subscribers to this area: @dotnet/interop-contrib Issue DetailsBackport of #91983 to release/8.0 Customer ImpactTestingRiskIMPORTANT: If this backport is for a servicing release, please verify that:
|
AaronRobinsonMSFT
added
the
Servicing-consider
|
@tannergooding or @stephentoub, Can I get a review here? |
AaronRobinsonMSFT
added
Servicing-approved
ghost
locked as resolved and limited conversation to collaborators
Backport of #91983 to release/8.0
/cc @AaronRobinsonMSFT
Customer Impact
This change is introducing new warnings that can impact users. See #91981.
Testing
Validate no warnings are emitted for
ref readonlywhen used with eitherinorref.Risk
None.
IMPORTANT: If this backport is for a servicing release, please verify that:
The PR target branch is
release/X.0-staging, notrelease/X.0.If the change touches code that ships in a NuGet package, you have added the necessary package authoring and gotten it explicitly reviewed.