Skip to content

Warn about unsafe blocks in unsafe functions - #6038

Closed
alexcrichton wants to merge 3 commits into
rust-lang:incomingfrom
alexcrichton:more-unsafe
Closed

Warn about unsafe blocks in unsafe functions#6038
alexcrichton wants to merge 3 commits into
rust-lang:incomingfrom
alexcrichton:more-unsafe

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

Because unsafe functions are never warned about, then all unsafe blocks in unsafe functions should definitely be warned about (no need to be redundant). This fixes this case, adds tests, cleans up remaining cases, and then fixes a few other import warnings being spit out.

bors added a commit that referenced this pull request Apr 24, 2013
Because unsafe functions are never warned about, then all `unsafe` blocks in unsafe functions should definitely be warned about (no need to be redundant). This fixes this case, adds tests, cleans up remaining cases, and then fixes a few other import warnings being spit out.
@borsbors closed this Apr 24, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Sep 24, 2020
Add `manual-strip` lint
Add `manual-strip` lint.
changelog: Add `manual-strip` lint
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
The proper fix I think is:
* move rust-lang/rust library crates to a separate workspace
* when packaging rust-src component, vendor sources of external deps
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
6294: Add a hacky remidy for rust-lang#6038 r=matklad a=matklad
bors r+
🤖
Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>
Sign up for freeto 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

@alexcrichton@pcwalton@bors