Skip to content
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

Document CorHandleWeakWinRT #42659

Merged
merged 1 commit into from
Sep 19, 2024
Merged

Document CorHandleWeakWinRT #42659

merged 1 commit into from
Sep 19, 2024

Conversation

noahfalk
Copy link
Member

@noahfalk noahfalk commented Sep 18, 2024

  • Add docs for that enumeration value in the .NET debugging APIs.
  • Also updates the CorReferenceFinalizer value. 80000002 in decimal is the correct value in the shipping code. Probably some earlier developer intended it to be hexadecimal in the product but made a mistake. Since it has already shipped it won't be changed.

Fixes dotnet/diagnostics#4171


Internal previews

📄 File 🔗 Preview link
docs/framework/unmanaged-api/debugging/corgcreferencetype-enumeration.md docs/framework/unmanaged-api/debugging/corgcreferencetype-enumeration

- Add docs for that enumeration value in the .NET debugging APIs.
- Also updates the CorReferenceFinalizer value. 80000002 in decimal is the correct value in the shipping code. Probably some earlier developer intended it to be hexadecimal in the product but made a mistake. Since it has already shipped it won't be changed.

Fixes dotnet/diagnostics#4171
@noahfalk noahfalk requested a review from a team as a code owner September 18, 2024 23:35
@dotnet-bot dotnet-bot added this to the September 2024 milestone Sep 18, 2024
@noahfalk
Copy link
Member Author

@tommcdon PTAL

@noahfalk
Copy link
Member Author

@gewarren - I see the CI failed, but if I understand it correctly the warning it is complaining about is in a different file that isn't touched by this PR.

@gewarren
Copy link
Contributor

gewarren commented Sep 19, 2024

@gewarren - I see the CI failed, but if I understand it correctly the warning it is complaining about is in a different file that isn't touched by this PR.

Ooh, yes, let me fix that.

Edit: Will be fixed by #42660.

@gewarren gewarren closed this Sep 19, 2024
@gewarren gewarren reopened this Sep 19, 2024
Copy link
Member

@tommcdon tommcdon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@noahfalk noahfalk merged commit bd6965b into dotnet:main Sep 19, 2024
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Please document CorGCReferenceType::CorHandleWeakWinRT
5 participants