-
Notifications
You must be signed in to change notification settings - Fork 5
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
Remove SignatureAdapter memory leak #1836
Conversation
app/src/main/java/com/infomaniak/mail/ui/newMessage/NewMessageFragment.kt
Outdated
Show resolved
Hide resolved
f9f0eb0
to
e785413
Compare
3b46d4a
to
56865cc
Compare
e785413
to
dba510d
Compare
56865cc
to
47a717b
Compare
dba510d
to
998a574
Compare
This PR/issue depends on:
|
998a574
to
8f5b285
Compare
627db89
to
8f4dcd3
Compare
8f5b285
to
cceb8f2
Compare
|
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.
Are we sure the adapter for a list actually leaks in the first place? Because maybe only recyclerview's ones do. Maybe list adapters don't have a strong reference to their ListPopupWindow
Depends on #1834
Remove strong reference about SignatureAdapter in NewMessageFragment.