Skip to content

Remove incorrect noop in unix stack pool management#13069

Merged
alexcrichton merged 1 commit intobytecodealliance:mainfrom
alexcrichton:remove-wrong-noop
Apr 13, 2026
Merged

Remove incorrect noop in unix stack pool management#13069
alexcrichton merged 1 commit intobytecodealliance:mainfrom
alexcrichton:remove-wrong-noop

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

This commit removes a call to commit_pages in the unix_stack_pool.rs file. This function is a noop on unix and the parameters passed in for this call were also incorrect if it actually needed to do something anyway. This is otherwise just distracting, so remove it.

This commit removes a call to `commit_pages` in the `unix_stack_pool.rs`
file. This function is a noop on unix and the parameters passed in for
this call were also incorrect if it actually needed to do something
anyway. This is otherwise just distracting, so remove it.
@alexcrichton alexcrichton requested a review from a team as a code owner April 13, 2026 15:30
@alexcrichton alexcrichton requested review from fitzgen and removed request for a team April 13, 2026 15:30
@fitzgen fitzgen added this pull request to the merge queue Apr 13, 2026
@github-actions github-actions Bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Apr 13, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 13, 2026
@alexcrichton alexcrichton added this pull request to the merge queue Apr 13, 2026
Merged via the queue into bytecodealliance:main with commit 42266d4 Apr 13, 2026
48 checks passed
@alexcrichton alexcrichton deleted the remove-wrong-noop branch April 13, 2026 22:01
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