----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128944/ -----------------------------------------------------------
(Updated Oct. 26, 2016, 2:54 a.m.) Status ------ This change has been discarded. Review request for KDE Frameworks. Repository: kcoreaddons Description ------- While analising plasmashell under heaptrack, one of the sore spots is temporary allocations within DesktopFileParser. This improves the situation by: * Only converting to QString/utf8 once. * Using QStringRef instead of fully splitting QString to parse them. Diffs ----- src/lib/plugin/desktopfileparser.cpp 2eb198d src/lib/plugin/desktopfileparser_p.h c61b297 Diff: https://git.reviewboard.kde.org/r/128944/diff/ Testing ------- tests still pass, plasma still works normally. heaptrack plasmashell: after: allocations: 4169312 leaked allocations: 83225 temporary allocations: 606902 before: allocations: 4680691 leaked allocations: 84825 temporary allocations: 819292 Thanks, Aleix Pol Gonzalez