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

Add Memo, Mutable #16

Closed
fredpi opened this issue Mar 19, 2018 · 2 comments
Closed

Add Memo, Mutable #16

fredpi opened this issue Mar 19, 2018 · 2 comments

Comments

@fredpi
Copy link
Contributor

fredpi commented Mar 19, 2018

I suggest adding the Memo Type to HandySwift, based on this repo.

My suggested implementation adjusting the implementation a bit can be found here.

@fredpi fredpi mentioned this issue Mar 19, 2018
@Jeehut
Copy link
Member

Jeehut commented Mar 22, 2018

Could you please explain, how this would improve Swift with some usage examples and why you think "it should have been part of the Swift Standard Library"? Also, could you check if there were discussions about this (or something solving the same problem) at the new Swift Evolution Forum and link it here, if you can find it? Thank you in advance.

@fredpi
Copy link
Contributor Author

fredpi commented Apr 8, 2018

As pointed out in #17, wrapper types aren't very swifty, in that they should be a part of the standard library. While Mutable would be needed for some rare use cases (as seen in the Memo implementation), its use shouldn't be encouraged by being a part of Swift itself.

Same goes for Memo, which, in fact, is just a lazy box somehow. So while useful, they both don't fit here. Closing...

@fredpi fredpi closed this as completed Apr 8, 2018
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

2 participants