clementine-player / Android-Remote

Control Clementine from your Android device
GNU General Public License v3.0
198 stars 80 forks source link

Memory Leak #185

Open Robigus92 opened 6 years ago

Robigus92 commented 6 years ago

After a double rotation of the ClementineSettings there are in heap memory 2 instances more of:

class de.qspool.clementineremote.ui.settings.PreferencesBehaviorPlayer @ 0x12d4b400 class de.qspool.clementineremote.ui.settings.PreferencesBehaviorLibrary @ 0x12d4c400 class de.qspool.clementineremote.ui.settings.PreferencesConnection @ 0x12d4ec00 class de.qspool.clementineremote.ui.settings.PreferencesInformationAbout @ 0x12d4f800 class de.qspool.clementineremote.ui.settings.PreferencesInformationAbout$3 @ 0x12d58340 class de.qspool.clementineremote.ui.settings.PreferencesInformationLicenses$1 @ 0x12d58580 class de.qspool.clementineremote.ui.settings.ClementineSettings$PreferenceListFragment @ 0x12cb5c00 class de.qspool.clementineremote.ui.settings.PreferencesInformationLicenses @ 0x12d51800 class de.qspool.clementineremote.ui.settings.PreferencesInformationLicenses$2 @ 0x12d58640 class de.qspool.clementineremote.ui.settings.ClementineSettings$PreferenceListFragment$1 @ 0x12d3e880 class de.qspool.clementineremote.ui.settings.PreferencesBehaviorDownloads @ 0x12d4d000 class de.qspool.clementineremote.ui.settings.PreferencesBehaviorAdvanced @ 0x12d4dc00

It should not be like this because the rotations return to the same GUI state and the memory should stay in the same state as well. It could be a symptom of a bad management of the onCreate-onDestroy lifecycle.

I have encountered this problem on an Android emulator running 7.1.1 os version.