D15134: decrease StateData space by more than 50% and half the number of needed mallocs
Volker Krause
noreply at phabricator.kde.org
Wed Aug 29 08:41:10 BST 2018
vkrause added a comment.
I can't answer the m_defData question without digging deeper into this unfortunately. The rest looks good to me though.
INLINE COMMENTS
> state.cpp:117
> return false;
> - return d->m_contextStack.top()->indentationBasedFoldingEnabled();
> + return d->m_contextStack.last().first->indentationBasedFoldingEnabled();
> }
last() has a non-const (detaching) overload, is there a risk we are hitting this here due to the d-> indirection? All other occurrences seem to be clearly on const members.
REPOSITORY
R216 Syntax Highlighting
REVISION DETAIL
https://phabricator.kde.org/D15134
To: cullmann, vkrause, dhaumann
Cc: kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, cullmann, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180829/147477dc/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list