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.

Sign up for GitHub

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

Jump to bottom

Allow casts wherever implict coercion is allowed #18601

Closed
nrc opened this issue Nov 4, 2014 · 1 comment · Fixed by #23630
Closed

Allow casts wherever implict coercion is allowed #18601

nrc opened this issue Nov 4, 2014 · 1 comment · Fixed by #23630
Labels
B-RFC-approved Blocker: Approved by a merged RFC but not yet implemented.

Comments

@nrc
Copy link
Member

nrc commented Nov 4, 2014

and add a lint for casts which are subtypes.

See RFC 401.

Part of #18469
@nrc nrc mentioned this issue Nov 4, 2014
Closed
@steveklabnik
Copy link
Member

Same here, isn't #18469 the issue for this?

@steveklabnik steveklabnik added the A-DSTs Area: Dynamically-sized types (DSTs) label Jan 27, 2015
@nrc nrc removed the A-DSTs Area: Dynamically-sized types (DSTs) label Mar 23, 2015
@nrc nrc mentioned this issue Mar 23, 2015
Merged
@steveklabnik steveklabnik added the B-RFC-approved Blocker: Approved by a merged RFC but not yet implemented. label Mar 23, 2015
bors added a commit that referenced this issue Mar 24, 2015
@bors
Auto merge of #23630 - nrc:coerce-tidy, r=<try>
ce76bff 
See notes on the first commit

Closes #18601

r? @nikomatsakis 

cc @eddyb
alexcrichton added a commit to alexcrichton/rust that referenced this issue Mar 24, 2015
@alexcrichton
rollup merge of rust-lang#23630: nrc/coerce-tidy
a1d2e62 
See notes on the first commit

Closes rust-lang#18601

r? @nikomatsakis

cc @eddyb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
B-RFC-approved Blocker: Approved by a merged RFC but not yet implemented.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make all coercion valid casts and add trivial cast lints nrc/rust
2 participants
@steveklabnik @nrc