Skip to content

Remove an accessor of unforced_current_thread#13759

Merged
alexcrichton merged 3 commits into
bytecodealliance:mainfrom
alexcrichton:remove-unforced-access
Jun 29, 2026
Merged

Remove an accessor of unforced_current_thread#13759
alexcrichton merged 3 commits into
bytecodealliance:mainfrom
alexcrichton:remove-unforced-access

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

Juggle some things around to remove an otherwise-needed promotion.

Follow-up from #13695

@alexcrichton alexcrichton requested a review from a team as a code owner June 29, 2026 16:51
@alexcrichton alexcrichton requested review from pchickey and removed request for a team June 29, 2026 16:51

@pchickey pchickey left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think the diff on resources.rs line 302 should be backed out? otherwise lgtm

Comment thread crates/wasmtime/src/runtime/vm/component/resources.rs Outdated
@alexcrichton alexcrichton enabled auto-merge June 29, 2026 17:24
@alexcrichton alexcrichton force-pushed the remove-unforced-access branch from d5ccce0 to ff03058 Compare June 29, 2026 17:49
@alexcrichton alexcrichton added this pull request to the merge queue Jun 29, 2026
@github-actions github-actions Bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Jun 29, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 29, 2026
@alexcrichton alexcrichton added this pull request to the merge queue Jun 29, 2026
Merged via the queue into bytecodealliance:main with commit e324ea4 Jun 29, 2026
52 checks passed
@alexcrichton alexcrichton deleted the remove-unforced-access branch June 29, 2026 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasmtime:api Related to the API of the `wasmtime` crate itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants