<table><tr><td style="">rjvbb 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/D9824" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>But I won't write mega patches like you seem to prefer.</p></blockquote>

<p>I don't. They happen when it's more cumbersome than self-evident to go back and break up a new feature in baby-step components and justify adding them one at a time.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I try to keep things as minimal as possible. And I also can't give you an ETA or guarantee on when I'll fix the rest. This patch solves one big issues, and the added benchmark lies the foundation for future work. Let's start from here and get going.</p></blockquote>

<p>Agreed. It's a start, but if you can do the main 2 backends with (more or less) the same fix that'd be great. Apparently you rarely encounter the QFSW backend in your world anyway, so the risk should be small that applying a possibly suboptimal fix for that backend will satisfy you enough to let things rest there.<br />
On the contrary, someone who does use that backend all the time might realise that improvements were possible and step in.</p>

<p>So please do measure the impact of your fix on performance with the QFSW backend.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R244 KCoreAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9824" rel="noreferrer">https://phabricator.kde.org/D9824</a></div></div><br /><div><strong>To: </strong>mwolff, dfaure, rjvbb, KDevelop<br /><strong>Cc: </strong>markg, Frameworks<br /></div>