Skip to content

Remove used_with_arg; add Grapefruit test suite#2539

Open
mkeeter wants to merge 2 commits into
masterfrom
mkeeter/remove-used-with-arg
Open

Remove used_with_arg; add Grapefruit test suite#2539
mkeeter wants to merge 2 commits into
masterfrom
mkeeter/remove-used-with-arg

Conversation

@mkeeter
Copy link
Copy Markdown
Collaborator

@mkeeter mkeeter commented May 27, 2026

We would like to minimize our usage of nightly features. The only place we use #[used(linker)] is in the test suite, and it's not necessary: we need both #[unsafe(no_mangle)] and pub static mut, but together they prevent TESTS from being compiled out of the image. It's a bit awkward that we now need an unsafe to access tests, but that's the cost of pub static mut.

@mkeeter mkeeter requested review from hawkw, jamesmunns and labbott May 27, 2026 17:51
Copy link
Copy Markdown
Member

@hawkw hawkw 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 to me!

Comment thread test/test-suite/src/main.rs
@mkeeter mkeeter force-pushed the mkeeter/remove-used-with-arg branch 3 times, most recently from 36f4cd5 to 5140032 Compare May 28, 2026 15:08
@mkeeter mkeeter force-pushed the mkeeter/remove-used-with-arg branch from 5140032 to 8a5bdb0 Compare May 29, 2026 15:18
@mkeeter mkeeter force-pushed the mkeeter/remove-used-with-arg branch from 8a5bdb0 to 0e7ce2f Compare May 29, 2026 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants