Skip to content

Conversation

brson
Copy link
Contributor

@brson brson commented Sep 26, 2013

No description provided.

@chris-morgan
Copy link
Member

Any chance changes to syntax files in src/etc could be included in this PR?

Unverified

This user has not yet uploaded their public signing key.
@brson
Copy link
Contributor Author

brson commented Sep 26, 2013

@chris-morgan done

Build problems fixed.

@bors bors closed this Sep 27, 2013
@bors bors merged commit c1d6429 into rust-lang:master Sep 27, 2013
@klutzy
Copy link
Contributor

klutzy commented Sep 30, 2013

cc #9467

djkoloski pushed a commit to djkoloski/rust that referenced this pull request Sep 21, 2022
Fixes rust-lang#9504

Compiler generated call `into_iter` nodes return empty substs
which we need when checking it's predicates. Handle this by
simply exitting when we encounter one. This change introduces
false negatives in place of the ICEs.
djkoloski pushed a commit to djkoloski/rust that referenced this pull request Sep 21, 2022
Fix ICE in `unnecessary_to_owned`

Fixes rust-lang#9504

Compiler generated call `into_future` nodes return empty substs which we need when checking it's predicates. Handle this by simply exitting when we encounter one. This change introduces false negatives in place of the ICEs.

changelog: [`unnecessary_to_owned`]: fix ICE
blyxyas pushed a commit to blyxyas/rust that referenced this pull request Jan 3, 2024
… r=Jarcho

Remove mitigations for incorrect node args

This change https://github.com/rust-lang/rust/pull/118420/files#r1419874371 adds a missing `write_args` to properly record node args for lang-item calls.

Thus, in the `unnecessary_to_owned` lint, this ensures that the `call_generic_args` extracted by `get_callee_generic_args_and_args` are always correct, and we can remove the mitigation for rust-lang#9504 and rust-lang#10021 since the root cause has been fixed.

I'm not sure if there is other now-unnecessary code that can be removed, but this is the one I found when investigating rust-lang/rust-clippy#11965 (comment).

changelog: none
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.

None yet

5 participants