-
Notifications
You must be signed in to change notification settings - Fork 29
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
Simplify lambdas, #164
Simplify lambdas, #164
Conversation
✅ Build kepka 1.0.134-uhytgggx completed (commit ab523b8325 by @Randl) |
Possibly part of #27 |
Resolve conflicts? |
✅ Build kepka 1.0.141-fwyihqvf completed (commit 6cb4980580 by @Randl) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but please squash commits with one-line descriptions and make the commit message more descriptive for last commit ("Backport ").
✅ Build kepka 1.0.142-eiefnpmu completed (commit 9627985072 by @Randl) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FnMut is not defined in our codebase. It seems that it's defined here in upstream: https://github.com/telegramdesktop/tdesktop/blob/c7a4d67cfb63af74f6da6ca35aea9ad08cf65588/Telegram/SourceFiles/core/basic_types.h#L30
Partially backport dd81f5d
✅ Build kepka 1.0.144-uncpnqnh completed (commit 4b5f73884c by @Randl) |
Backport