Library Scanner eats up memory
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mixxx |
Fix Released
|
Critical
|
Max Linke |
Bug Description
I noticed today that mixxx uses a large amount of memory while scanning for files (more then 5GB on my machine). Once the scanner is done the memory usage drops back to a more reasonable amount. This behavior exists since we merged the cover-art branch.
The reason for this is that the TIO objects now consume a lot more memory because they also store the cover images found while they are parsed and that we use `QObject:
http://
Using a debug statement to check when a TIO is deleted shows that for the libraryscanner they are all deleted once the scanner finishes. Removing from `DeleteLater` from the TIO construction solves the memory issues for me and I see that the TIO's are deleted during the scan.
To double check my results I remove the `CoverArt` private variable from TIO and replaced the getter objects to return default classes. This also solves the memory "leak".
My current solution would be to delete the TIO's in the scanner as soon as they are out of scope. I don't know why we use `DeleteLater` in the scanner anyway. I didn't notice anything not working.
Changed in mixxx: | |
status: | Confirmed → Fix Committed |
Changed in mixxx: | |
assignee: | nobody → Max Linke (max-linke) |
Changed in mixxx: | |
status: | Fix Committed → Fix Released |
https:/ /github. com/mixxxdj/ mixxx/pull/ 377