Uh oh!
There was an error while loading. Please reload this page.
Clarify first use of "elide" in lifetime.md - #28842
Conversation
rust-highfive
commented
Oct 5, 2015
Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @pcwalton (or someone else) soon. If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes. Please see the contribution instructions for more information. |
Gankra
commented
Oct 5, 2015
"elide" is standard community terminology for this family of behaviours, fwiw. I think in this case, this is an improvement though. @bors r+ rollup |
bors
commented
Oct 5, 2015
📌 Commit ac49eca has been approved by |
steveklabnik
commented
Oct 5, 2015
@bors: r- |
steveklabnik
commented
Oct 5, 2015
So: I do agree that 'elide' is a weird word, but we also call it "lifetime elision" everywhere, and so, in some sense, using this word aids discover-ability. So I'd like to talk about it before we merge. I'm not sure what the right path of action here is, exactly. |
Gankra
commented
Oct 5, 2015
I would add a parenthetical mentioning the concept of "elision" here if that's really desirable. I'd need to go over this section more to get the context, but I was under the impression that this was at a point where we're "not even ready" to discuss lifetime elision in earnest. |
corincerami
commented
Oct 5, 2015
That sounds like a reasonable idea. I think it makes sense to at least clarify the use of "elide", even if that just involves including something like "(see 'Lifetime Elision' below)" to the sentence. |
steveklabnik
commented
Oct 5, 2015
👍 to both things. I think a forward reference here is just fine. Maybe something like
? |
corincerami
commented
Oct 5, 2015
👍 to that idea. |
steveklabnik
commented
Oct 5, 2015
Great! Amend this PR and we can |
corincerami
commented
Oct 5, 2015
|
There was a problem hiding this comment.
can you wrap this to 80 chars please?
steveklabnik
commented
Oct 6, 2015
@bors: r+ rollup |
bors
commented
Oct 6, 2015
📌 Commit f490f51 has been approved by |
bors
commented
Oct 6, 2015
bors
commented
Oct 6, 2015
corincerami
commented
Oct 6, 2015
🌈 |
…ision_link, r=alexcrichton This link was added in rust-lang#28842 but doesn't work at https://doc.rust-lang.org/nightly/book/lifetimes.html. What works in my markdown preview doesn't work live, and vice versa.
No description provided.