Skip to content

Added command-line argument support for wasm32-wali-linux-musl#156062

Open
arjunr2 wants to merge 1 commit intorust-lang:mainfrom
arjunr2:wali-support-clargs
Open

Added command-line argument support for wasm32-wali-linux-musl#156062
arjunr2 wants to merge 1 commit intorust-lang:mainfrom
arjunr2:wali-support-clargs

Conversation

@arjunr2
Copy link
Copy Markdown
Contributor

@arjunr2 arjunr2 commented May 1, 2026

No description provided.

@rustbot rustbot added O-linux Operating system: Linux S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 1, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 1, 2026

r? @nia-e

rustbot has assigned @nia-e.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @ChrisDenton, libs
  • @ChrisDenton, libs expanded to 8 candidates
  • Random selection from Mark-Simulacrum, jhpratt, nia-e

@rustbot rustbot added the T-libs Relevant to the library team, which will review and decide on the PR/issue. label May 1, 2026
Copy link
Copy Markdown
Member

@nia-e nia-e left a comment

Choose a reason for hiding this comment

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

Looks good modulo one nitpick, thanks!

View changes since this review

CString::from_raw(ptr)
};
let mut arg_buf = arg_buf.into_bytes_with_nul();
let _ = arg_buf.pop();
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why not just use CString::into_bytes directly?

@nia-e nia-e added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

O-linux Operating system: Linux S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants