Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cleanup warnings #6690

Closed
wants to merge 10 commits into from
Closed

Cleanup warnings #6690

wants to merge 10 commits into from

Conversation

erickt
Copy link
Contributor

@erickt erickt commented May 23, 2013

Simple patch series to fix up all the warnings a rustc compile is giving at the moment. It also fixes a NOTE in to_bytes.rs to remove the to_bytes::iter_bytes_<N> functions.

@brson
Copy link
Contributor

brson commented May 23, 2013

Thanks for doing cleanup!

bors added a commit that referenced this pull request May 24, 2013
Simple patch series to fix up all the warnings a rustc compile is giving at the moment. It also fixes a NOTE in `to_bytes.rs` to remove the `to_bytes::iter_bytes_<N>` functions.
@bors bors closed this May 24, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Feb 11, 2021
Downgrade trivial_regex to nursery

See rust-lang#6690. I think there is still value in a trivial_regex lint, but only if clippy can tell that the regex is only ever constructed and applied to a single input.

```rust
let regex = Regex::new("trivial_regex")?;
println!("{}", regex.is_match(s));
// `regex` never used again
```

---

changelog: remove `trivial_regex` from default set of enabled lints
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