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

Nonparametric dropck (tracking issue for RFC 1238) #28498

Open
nikomatsakis opened this Issue Sep 18, 2015 · 5 comments

Comments

Projects
None yet
7 participants
@nikomatsakis
Contributor

nikomatsakis commented Sep 18, 2015

Tracking issue for rust-lang/rfcs#1238.

  • subtask: #[may_dangle] attribute #34761
  • can't close this until support for we have stable support for 3rd party collection types

cc @pnkfelix

@nikomatsakis

This comment has been minimized.

Contributor

nikomatsakis commented Sep 18, 2015

triage: P-high

bors added a commit that referenced this issue Oct 10, 2015

Auto merge of #28861 - pnkfelix:fsk-nonparam-dropck-issue28498, r=ari…
…elb1

implement RFC 1238: nonparametric dropck.

cc #28498 

cc @nikomatsakis
@nikomatsakis

This comment has been minimized.

Contributor

nikomatsakis commented Oct 29, 2015

triage: P-medium

@rust-highfive rust-highfive added P-medium and removed P-high labels Oct 29, 2015

@alexcrichton

This comment has been minimized.

Member

alexcrichton commented Feb 18, 2016

I think this was since implemented, so closing (but please reopen if there's something left!)

@SimonSapin

This comment has been minimized.

Contributor

SimonSapin commented May 4, 2016

I think this should be reopened with the B-unstable label. The RFC is implemented, but the #[unsafe_destructor_blind_to_params] attribute is still behind the dropck_parametricity feature gate which points to this as its tracking issue.

#[unsafe_destructor_blind_to_params] is used for Vec and a number of other types defined in the standard library. It isn’t any less useful for types outside the standard library. I’ve needed it for an arena allocator.

@nikomatsakis

This comment has been minimized.

Contributor

nikomatsakis commented May 4, 2016

I agree.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment