----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127763/#review94948 -----------------------------------------------------------
could you also have a look at it with heaptrack? it should instantly give you a total number of allocations after finish, which should already give us an indication whether this brings lots to the table. Your callgrind numbers seem to indicate so though. I just feel a bit uneasy with lazy-constructing the string without caching it. If that is happening a lot, you may want to cache the result - Milian Wolff On April 27, 2016, 1:10 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/127763/ > ----------------------------------------------------------- > > (Updated April 27, 2016, 1:10 p.m.) > > > Review request for KDE Frameworks and Christoph Feck. > > > Repository: kiconthemes > > > Description > ------- > > Leverages the implicitly shared nature of QString by only constructing the > paths when it's strictly necessary. It souldn't show a big performance impact > since the generated paths were only intermediate in most cases after all. > > This is especially important now that plasma has several KIconLoader > instances at the same time, but it should have an impact on every application > using kiconthemes (or under the plasma-integration effects). > > > Diffs > ----- > > src/kicontheme.cpp 735ecf7 > > Diff: https://git.reviewboard.kde.org/r/127763/diff/ > > > Testing > ------- > > Tests still pass, apps still work. > > Starting and closing dolphin under callgrind showed 1640M instructions, now > 1466M(10% improvement). > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
