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.

Sign up for GitHub

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

Jump to bottom

remove useless [unowned self] #529

Open

Akhrameev wants to merge 1 commit into Swinject:master from Akhrameev:master

Akhrameev commented Mar 10, 2023

as closures are @NoEscape by default and don't retain 'self' since Swift version 3

More info:
https://stackoverflow.com/questions/38141298/lazy-initialisation-and-retain-cycle
https://github.com/apple/swift-evolution/blob/main/proposals/0103-make-noescape-default.md
https://michael-kiley.medium.com/why-your-lazy-vars-arent-creating-strong-reference-cycles-in-ios-d512ff2c9403


          remove useless [unowned self]

0537a82

as closures are @NoEscape by default and don't retain 'self'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet