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

vector cleanup and turning ast::spanned into a struct #4380

Closed
wants to merge 10 commits into from

Conversation

erickt
Copy link
Contributor

@erickt erickt commented Jan 8, 2013

Just some code I have queued up in my repository I'd like to upstream. I'm converting all the #[auto_encode]ed code to use structs, so this is the first step. Plus it includes some vector cleanup and a function to partition vectors.

@graydon
Copy link
Contributor

graydon commented Jan 8, 2013

r+, what's the performance impact of the inlining?

@erickt erickt closed this Jan 14, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jul 14, 2020
…1995

Improvements for `type_repetition_in_bounds` lint

Some improvements for `type_repetition_in_bounds`:
- add a configurable threshold to trigger the lint (rust-lang#4380). The lint won't trigger anymore if there are more bounds (strictly) than `conf.max_trait_bounds` on this type.
- take generic args into account over bounded type (rust-lang#4323)
- don't lint for predicates generated in macros (rust-lang#4326)

Fixes rust-lang#4380,
Fixes rust-lang#4323,
Fixes rust-lang#4326,
Closes rust-lang#3764

changelog: Fix multiple FPs in `type_repetition_in_bounds` and add a configuration option

Note: the rust-lang#3764 has already been fixed but not closed
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.

2 participants