Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Don't cache debuginfo sessions #1534

Merged
merged 2 commits into from
Dec 17, 2021
Merged

Conversation

ranweiler
Copy link
Member

Remove the currently-unused DebugInfo.session field to ease async usage.

The removed field has type ObjectDebugSession, which is not Send. This is problematic in an async context. We could work around it, and probably even craft a Send wrapper type (which would require judicious use of unsafe). However, in our coverage task, we don't actually use debug sessions at all. In the future, we can still easily get them on-demand from Object values (which we do cache). So, just remove the cached session field until we actually need it.

@ranweiler ranweiler merged commit 97f04c2 into microsoft:main Dec 17, 2021
@ranweiler ranweiler deleted the dbg-session branch December 17, 2021 20:59
@ghost ghost locked as resolved and limited conversation to collaborators Jan 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants