Kiwix-android: NestedCoordinatorLayout is leaking

Created on 17 Nov 2020  路  8Comments  路  Source: kiwix/kiwix-android

Describe the bug
NestedCoorinatorLayout is leaking

Expected behavior
Memory leak shouldn't happen

Steps to reproduce the behavior:

  • Open Library Fragment
  • Tap the share button on the upper right corner
  • Search the wifi
  • Go back until you reach the home screen

Screenshots
Screenshot 2020-11-17 at 4 47 24 PM

Environment

  • Version of Kiwix Android : 3.4
  • Device : realme ,pixel
  • OS version : 10
bug

All 8 comments

@gouri-panda What do you think if we have a parent issue for memory leaks as well from where we can further reference all the memory leak issues? I mean quite similar to that of what we have for converting Java files to kotlin, doing so will help keeping track of leaks fixed. Thoughts?

I am fine with an issue per leak, the kotlin conversion task was just a way of allowing autonomous work by new contributors.

yes, that's a fantastic idea! I will leave it to @macgills @kelson42

Okay, I meant a big issue and creating minor issues like "this part is leaking" and referencing the major issue in the smaller ones, just an idea, nothing else

We don't want an eternal task ever, ideally we eventually get this repo down to 0 open issues. We should only go multi ticket when we are decomposing a very large yet finite task and even then tags are probably a better mechanism to achieve this. When a leak is detected we should create a ticket to fix that leak and I don't particularly see the benefit of having a history of our leaks fixed referenceable via a ticket. Like we don't have a "fix bugs" ticket. Hopefully this gives an insight into general ticket philosophy at kiwix

Now this makes it as clear as today's sky.

@macgills I can't reproduce this issue and #2502 in the recent develop branch. should we close this?

Close 'em

Was this page helpful?
0 / 5 - 0 ratings

Related issues

danielzgtg picture danielzgtg  路  4Comments

sonusourav picture sonusourav  路  5Comments

kelson42 picture kelson42  路  5Comments

Popolechien picture Popolechien  路  4Comments

RaiLokesh picture RaiLokesh  路  4Comments