| Summary: | theoretical configwidget leak in services | ||
|---|---|---|---|
| Product: | [Applications] amarok | Reporter: | Harald Sitter <sitter> |
| Component: | general | Assignee: | Amarok Bugs <amarok-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | ||
| Priority: | NOR | ||
| Version First Reported In: | 2.5.90 (2.6 beta) | ||
| Target Milestone: | 2.6 | ||
| Platform: | unspecified | ||
| OS: | Linux | ||
| Latest Commit: | http://commits.kde.org/amarok/75348119fe891ceb8f9c44285847c77103dfaf1f | Version Fixed/Implemented In: | 2.8 |
| Sentry Crash Report: | |||
Git commit 75348119fe891ceb8f9c44285847c77103dfaf1f by Ralf Engels. Committed on 20/02/2013 at 21:29. Pushed by rengels into branch 'master'. Fix: theoretical configwidget leak in services Fix memory leak. Cleanup additional layouts and widgets Remove KDE logging. We have our own logging FIXED-IN: 2.8 M +3 -5 src/services/amazon/AmazonSettingsModule.cpp M +1 -10 src/services/ampache/AmpacheSettings.cpp M +3 -4 src/services/gpodder/GpodderServiceSettings.cpp M +1 -0 src/services/lastfm/LastFmServiceSettings.cpp M +2 -9 src/services/magnatune/MagnatuneSettingsModule.cpp M +2 -10 src/services/mp3tunes/Mp3tunesSettingsModule.cpp http://commits.kde.org/amarok/75348119fe891ceb8f9c44285847c77103dfaf1f |
all service kcms fail to actively recover their configwidget memory in the KCMs > m_configDialog = new Ui::Mp3tunesConfigWidget; while in practice this is no issue right now (KCM being a runtime loaded plugin it's memory is recovered when the KCM is destroyed) it would be good to have the widgets deleted as otherwise a possible switch away from KCMing the config would lead to an active leak Reproducible: Always