jtamate added a comment.
In D12897#276182 <https://phabricator.kde.org/D12897#276182>, @dhaumann wrote: > @mwolff To me this looks ok - do you see an issue with this? E.g. that KTextEditor will require much more memory for almost no gain? If there is a more constrained limit, this should be changed, of course. > I wonder if that really makes sense, given not all blocks contain ranges at all and they not really correlate with the block size. > Does that change really make it much faster or just shift the costs? In my tests it doen't really make it much faster, but now the top cost in undo is really the undo method, not QHash memory management. 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
