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

Stabilization of weak pointers #27465

Closed
milibopp opened this issue Aug 2, 2015 · 3 comments
Closed

Stabilization of weak pointers #27465

milibopp opened this issue Aug 2, 2015 · 3 comments

Comments

@milibopp
Copy link
Contributor

milibopp commented Aug 2, 2015

Currently, both alloc::rc::Weak and alloc::arc::Weak are unstable. The somewhat vague reason for this is that weak pointers may not belong in this module. Now this has been addressed multiple times in #24028 and I have been observing this situation for a few months. However, as far as I can tell, nothing has happened so far.

Stabilizing my FRP library Carboxyl is currently blocked on this issue (see milibopp/carboxyl#47). Hence, I would really, really love to see this stabilized.

So my main question there is: is it really, as the instability note indicates, just a matter of discussing whether it should be moved to another module? Or is there anything more to it? In either case I would be happy to help out with anything that needs doing.

I did some archeology on the git logs. And, apparently @brson marked alloc::rc::Weak as unstable a year ago in cf8bfde. Shortly thereafter, @alexcrichton adapted this policy to alloc::arc::Weak in e5da6a7. So perhaps one of you could comment on the motivation behind that and whether it still applies today?

@bluss
Copy link
Member

bluss commented Aug 2, 2015

See the active discussion at rust-lang/rfcs#1232

@milibopp
Copy link
Contributor Author

milibopp commented Aug 2, 2015

Thank you for the hint. I had not picked up that RFC yet.

@alexcrichton
Copy link
Member

Yeah the RFC @bluss linked is where discussion is happening now so I'm going to close this in favor of that

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

No branches or pull requests

3 participants