dhaumann created this revision. dhaumann added a reviewer: cullmann. Herald added projects: Kate, Frameworks. Herald added subscribers: kde-frameworks-devel, kwrite-devel. dhaumann requested review of this revision.
REVISION SUMMARY Instead of creating N empty QIcons, use just one QIcon that internally is implicitly shared, see: https://doc.qt.io/qt-5/implicit-sharing.html TEST PLAN make REPOSITORY R39 KTextEditor BRANCH use-qicon-implicitly-shared REVISION DETAIL https://phabricator.kde.org/D24166 AFFECTED FILES src/mode/katemodemenulist.cpp To: dhaumann, cullmann Cc: kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann