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

feat: add exercise involving lifetime elision #1719

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sgasse
Copy link

@sgasse sgasse commented Oct 14, 2023

No description provided.

@sgasse
Copy link
Author

sgasse commented Oct 14, 2023

Hey @shadows-withal ! Your talk yesterday at EuroRust inspired me to contribute this example 🙂 A colleague of mine asked for help with a much more cluttered real-life example of the same problem that he ran into. Hope it can be useful to others as well.

exercises/lifetimes/lifetimes4.rs Outdated Show resolved Hide resolved
info.toml Outdated Show resolved Hide resolved
@sgasse sgasse force-pushed the sgasse/feature/lifetime_elision_example branch from 0212875 to f3729f1 Compare October 16, 2023 18:31
@sgasse sgasse force-pushed the sgasse/feature/lifetime_elision_example branch from f3729f1 to 5adf590 Compare April 19, 2024 15:23
@sgasse
Copy link
Author

sgasse commented Apr 19, 2024

I integrated your suggestions and updated the wording to be more concrete. Could you take another look @shadows-withal ?

@sgasse
Copy link
Author

sgasse commented Apr 22, 2024

Or maybe @mo8it , could you take a look?

@mo8it
Copy link
Contributor

mo8it commented Apr 22, 2024

Hi @sgasse
Currently, my priority is to release version 6. I will get back to this PR afterwards. My estimate is 1-2 weeks.

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.

None yet

3 participants