Skip to content

Conversation

@thestinger
Copy link
Contributor

No description provided.

bors added a commit that referenced this pull request Aug 4, 2013
@bors bors closed this Aug 4, 2013
@alexcrichton
Copy link
Member

props to @thestinger for this awesome work!

flip1995 pushed a commit to flip1995/rust that referenced this pull request Apr 7, 2022
Rework `undocumented_unsafe_blocks`

fixes: rust-lang#8264
fixes: rust-lang#8449

One thing came up while working on this. Currently comments on the same line are supported like so:

```rust
/* SAFETY: reason */ unsafe {}
```

Is this worth supporting at all? Anything other than a couple of words doesn't really fit well.

edit: [zulip topic](https://rust-lang.zulipchat.com/#narrow/stream/257328-clippy/topic/.60undocumented_unsafe_blocks.60.20same.20line.20comment)

changelog: Don't lint `undocumented_unsafe_blocks` when the unsafe block comes from a proc-macro.
changelog: Don't lint `undocumented_unsafe_blocks` when the preceding line has a safety comment and the unsafe block is a sub-expression.
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