broulik created this revision. broulik added a reviewer: Plasma. Restricted Application added projects: Plasma, Frameworks. Restricted Application added subscribers: Frameworks, plasma-devel.
REVISION SUMMARY Documentation says "Children are stored internally as a linked list, so iterating over the children via the index is suboptimal." TEST PLAN Verified that it doesn't crash and the number of repositioned nodes stays identical, ie. I didn't miss one. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D8046 AFFECTED FILES src/declarativeimports/core/framesvgitem.cpp To: broulik, #plasma Cc: plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart