Skip to content
This repository was archived by the owner on Sep 8, 2025. It is now read-only.

Remove no-longer-needed helper method#182

Merged
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:remove-helper-method
Jun 3, 2025
Merged

Remove no-longer-needed helper method#182
alexcrichton merged 1 commit into
bytecodealliance:mainfrom
alexcrichton:remove-helper-method

Conversation

@alexcrichton
Copy link
Copy Markdown
Member

In trying to upstream this helper method it had surprising semantics for insert-beyond-the-end and so upon investigating I concluded that this may be a historical artifact at this point and no longer needed. To reduce the diff with upstream this commit removes the method and replaces the one usage with a locally-equivalent structure.

In trying to upstream this helper method it had surprising semantics for
insert-beyond-the-end and so upon investigating I concluded that this
may be a historical artifact at this point and no longer needed. To
reduce the diff with upstream this commit removes the method and
replaces the one usage with a locally-equivalent structure.
@alexcrichton alexcrichton added this pull request to the merge queue Jun 3, 2025
Merged via the queue into bytecodealliance:main with commit 839d7f2 Jun 3, 2025
42 checks passed
@alexcrichton alexcrichton deleted the remove-helper-method branch June 3, 2025 16:38
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