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

[RFC issue 1468] make some common traits object-safe #31193

Closed
wants to merge 6 commits into from

Conversation

durka
Copy link
Contributor

@durka durka commented Jan 25, 2016

This is a very dubious change but @eddyb was curious about the fallout. @brson crater this too?

cc rust-lang/rfcs#1468

durka added 4 commits January 25, 2016 17:39
specifically: Clone, Default, From, Into, FromStr, One, Zero

breakage: causes RFC 1214 and other WF warnings/errors

[breaking-change]
@rust-highfive
Copy link
Collaborator

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@nikomatsakis
Copy link
Contributor

I'm confused -- do you really want to land this, or just want the results from a crater run?

@durka
Copy link
Contributor Author

durka commented Jan 26, 2016

I think for now just the results of a crater run would inform the discussion on that issue. Certainly it would be nice for more things to be object-safe, but there was fallout even within the tests and compiler so it kinda seems like a non-starter unless we have super compelling reasons.

@nikomatsakis
Copy link
Contributor

@durka

Certainly it would be nice for more things to be object-safe

Hmm, can you say more about this? In particular, if the trait can't actually be used with an object, then why would it be nice for it to be object safe?

@durka
Copy link
Contributor Author

durka commented Jan 26, 2016

@nikomatsakis
Copy link
Contributor

OK, well, I've kicked off a crater run (I don't think @brson saw the ping). I'll let you know what it says.

@nikomatsakis
Copy link
Contributor

Crater report: https://gist.github.com/nikomatsakis/6478dde577e9affd957d

Summary: 3702 crates tested: 1984 working / 1160 broken / 305 regressed / 1 fixed / 252 unknown.

@durka durka closed this Jan 31, 2016
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