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

Extend allocation lint for boxing expressions #10881

Merged
merged 1 commit into from
Dec 9, 2013

Conversation

sanxiyn
Copy link
Member

@sanxiyn sanxiyn commented Dec 9, 2013

No description provided.

@alexcrichton
Copy link
Member

Amazing! Could you rebase these commits into one patch as well? Other than that r=me

@sanxiyn
Copy link
Member Author

sanxiyn commented Dec 9, 2013

Squashed.

@bors bors closed this Dec 9, 2013
@bors bors merged commit 3b14f25 into rust-lang:master Dec 9, 2013
@sanxiyn sanxiyn deleted the allocation-lint-2 branch December 9, 2013 18:36
lnicola pushed a commit to lnicola/rust that referenced this pull request Aug 23, 2022
…ses, r=Veykril

feat: Add an assist for inlining all type alias uses

## Description
`inline_type_alias_uses` assist tries to inline all selected type alias occurrences.

### Currently
Type alias used in `PathType` position are inlined.

### Not supported
- Removing type alias declaration if all uses are inlined.
- Removing redundant imports after inlining all uses in the file.
- Type alias not in `PathType` position, such as:
  - `A::new()`
  - `let x = A {}`
  - `let bits = A::BITS`
  - etc.

## Demonstration

![example](https://user-images.githubusercontent.com/45790125/184905226-9cb8ac81-1439-4387-a13b-e18ad4ecf208.gif)

## Related Issues
Partially fixes rust-lang#10881
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jun 30, 2023
…p, r=llogiq

[`useless_conversion`]: pluralize if there are multiple `.into_iter()` calls

context: rust-lang/rust-clippy#10814 (comment)

changelog: [`useless_conversion`]: pluralize if there are multiple `.into_iter()` calls in the chain

r? `@llogiq`
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