D10670: Reduce plasmashell frozen time

Jaime Torres Amate noreply at phabricator.kde.org
Tue Feb 20 07:24:25 UTC 2018


jtamate added a comment.


  In D10670#209823 <https://phabricator.kde.org/D10670#209823>, @mwolff wrote:
  
  > ok, looking at the reasoning in the other commit:
  >
  > - you need to extend the commit message here
  
  
  Will be done.
  
  > - you need to provide a comment in the code the clarifies what's going on here
  
  Will be done.
  
  > in general, I don't see how such a comparison can be so costly - the real problem would be too many native events, no? did you maybe profile a debug build or something?
  
  A qstrcmp is more expensive than a single cpu instruction in i586 builds (debug or release), unless it uses SSE4 instruction in the x64 world.
  
  Unfortunately I haven't found what causes so many native events.

REPOSITORY
  R242 Plasma Framework (Library)

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

To: jtamate, #frameworks, #plasma, mwolff
Cc: mwolff, plasma-devel, davidedmundson, michaelh, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180220/d07d3399/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list