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

identify traits in libstd that should use associated lifetimes #17828

Closed
pnkfelix opened this issue Oct 6, 2014 · 4 comments
Closed

identify traits in libstd that should use associated lifetimes #17828

pnkfelix opened this issue Oct 6, 2014 · 4 comments
Assignees
Labels
P-medium Medium priority

Comments

@pnkfelix
Copy link
Member

pnkfelix commented Oct 6, 2014

as a subtask of #17307, once the associated lifetimes part of associated items is implemented, we will want to update libstd to use it.

My google doc from #17826 includes a column for associated lifetimes; I copied the link again below.

Most/all of the cases are related to slice traits, and its possible that they will just go away with other design changes. I am not sure.

https://docs.google.com/spreadsheets/d/1Y_NAgzEK3FWcG0LjcykyKJEAibSFI3w-v-ekZ4GU8n4/edit?usp=sharing

@pnkfelix
Copy link
Member Author

pnkfelix commented Oct 7, 2014

Nominating.

I had thought this would not block 1.0, but @aturon points out here that we do not intend for lifetimes to be used as "input" parameters on a trait, but we still need to associate them and refer to them.

So this is probably P-backcompat-libs. And depending on what libraries are affected, it may also be on the 1.0 milestone.

@aturon
Copy link
Member

aturon commented Oct 9, 2014

@pnkfelix I added a comment to the subtask for implementing associated lifetimes to clarify how I see things.

@pnkfelix
Copy link
Member Author

pnkfelix commented Oct 9, 2014

Assigning P-high, not 1.0 milestone.

@pnkfelix pnkfelix added P-medium Medium priority and removed I-nominated labels Oct 9, 2014
@aturon
Copy link
Member

aturon commented Jan 8, 2015

Closing; due to DST, no traits in std actually need this.

@aturon aturon closed this as completed Jan 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-medium Medium priority
Projects
None yet
Development

No branches or pull requests

2 participants