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

refactor: KeyWeakMap cleanup #41253

Merged

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Feb 6, 2024

Backport of #41242

See that PR for details.

Notes: none

trop bot and others added 7 commits February 6, 2024 15:18
Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
no inheritance used, so no need for virtual dtor?

Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
Co-authored-by: Charles Kerr <charles@charleskerr.com>
@trop trop bot requested a review from ckerr February 6, 2024 15:19
@trop trop bot mentioned this pull request Feb 6, 2024
3 tasks
@trop trop bot added 28-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Feb 6, 2024
@ckerr ckerr merged commit a4f6d8a into 28-x-y Feb 6, 2024
17 checks passed
@ckerr ckerr deleted the trop/28-x-y-bp-refactor-keyweakmap-cleanup-1707232701804 branch February 6, 2024 18:43
Copy link

release-clerk bot commented Feb 6, 2024

No Release Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
28-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant