D14408: SymbolView: Avoid unneeded update of current item

[Kåre Särs]

sars added a comment.


  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.
  
  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.
  
  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.

REPOSITORY
  R40 Kate

REVISION DETAIL
  https://phabricator.kde.org/D14408

To: loh.tar, #kate, sars
Cc: ngraham, sars, kwrite-devel, #kate, michaelh, kevinapavew, demsking, cullmann, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180727/a85d4f5e/attachment.html>