D12897: Reserve space for the cachedLineForRanges Qhash (optimization)

Jaime Torres Amate noreply at phabricator.kde.org
Tue Jun 12 07:28:16 UTC 2018


jtamate abandoned this revision.
jtamate added a comment.


  More numbers:
  The patch reserves 67 buckets in the QHash table vs. 37 by default.
  In my test, pasing the content of https://www.boe.es/legislacion/documentos/ConstitucionCASTELLANO.pdf from Okular,
  the number of items in the cache is a maximum of 24 members when inserting.
  But the first undo uses up to 2300 members (4099 buckets), and later undoes only use a maximum of 43 members.
  
  The performance problem is in the first undo. What could be the difference between the first undo and subsequent undoes?

REPOSITORY
  R39 KTextEditor

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

To: jtamate, #kate, #frameworks
Cc: cullmann, dhaumann, mwolff, kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180612/e19c0db1/attachment.html>


More information about the KWrite-Devel mailing list