<table><tr><td style="">sars 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/D14408">View Revision</a></tr></table><br /><div><div><p>You cannot measure the CPU consumption of this particular function by looking at the "System Activity" while scrolling up and down the file. There are a million other code paths that get executed while you scroll the page. System Activity gives you an average CPU load of the application during 2000ms. This less than 1ms function only gets called once when you start scrolling and once when you stop.</p>

<p>What this patch does is that it avoids a for-loop of normally less than 100 elements, if you only scroll a couple of lines at a time.</p>

<p>Sorry, I'm not convinced before I get a trace from a profiler like hotspot, that clearly indicates that this patch would have any noticeable effect.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14408">https://phabricator.kde.org/D14408</a></div></div><br /><div><strong>To: </strong>loh.tar, Kate, sars<br /><strong>Cc: </strong>ngraham, sars, kwrite-devel, Kate, michaelh, kevinapavew, demsking, cullmann, dhaumann<br /></div>