<table><tr><td style="">apol added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D14162">View Revision</a></tr></table><br /><div><div><p>Correct, I wanted to fix all the allocations as the commit message says, then I realised it wasn't even working well when I added the unit test, so I fixed that too.</p>

<p>The problem with the allocation is that it's allocating for the remaining part of the field and freeing again the rest every time we read each element of the list. This showed up in heaptrack for Discover.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R237 KConfig</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14162">https://phabricator.kde.org/D14162</a></div></div><br /><div><strong>To: </strong>apol, Frameworks, dfaure<br /><strong>Cc: </strong>dfaure, anthonyfieroni, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>