Apps-android-commons: Leaks needs to be solved before release

Created on 26 Mar 2018  路  10Comments  路  Source: commons-app/apps-android-commons

Summary:

Here are leak reports needs to be solved before release (means this week)

Steps to reproduce:

Play with app, in general.

Add System logs:

leaks

This issue can be solved by several people. Please use comments efficiently to assign it yourselves by using letters (A/B/C/D/E/F).

Besides, this issue needs to be done in a week so help is appreciated :)

good first issue high priority nearby pre-release

Most helpful comment

I'll kick things off by taking up the the 'A' leak. I can try to fix all of them too if that's okay. 馃憤 @nicolas-raoul @neslihanturan

All 10 comments

I'll kick things off by taking up the the 'A' leak. I can try to fix all of them too if that's okay. 馃憤 @nicolas-raoul @neslihanturan

Firstly you can start with 'A', since this issue is urgent and requires parallel working. If you are done with 'A' don't hesitate to get another one, after you wrote comment here @yashshah7197

Thanks for volunteering to do "A", @yashshah7197 . :) If anyone else would like to take the others, please feel free to mention it here.

@neslihanturan can you tell me how I could reproduce these leaks? LeakCanary doesn't seem to be detecting anything for me yet.

On a side note, sorry for the late updates. I'm down with a bit of fever and cough so can't really work on anything properly right now. 馃檲

@yashshah7197 just playing with app, I don't know either. I hope you get well soon.

I am working on B and C

@yashshah7197 this issues needs to be done today, will you be able to make it up? If not I will give a try.

I am on D,E,F

It seems like E and F is something we can ignore: https://medium.com/@amitshekhar/android-memory-leaks-inputmethodmanager-solved-a6f2fe1d1348

Closing, we can create a new issue if there are further leaks. :)

Was this page helpful?
0 / 5 - 0 ratings