kossebau added inline comments. INLINE COMMENTS
> kossebau wrote in normalvimode.cpp:3844 > Ah, no, textLines.last().length() needs the string section, so just a plain > count would not be enough, only if combining with a QString::section call > later, I now remember. > And some other patch simply turned this into splitRef, so just one alloc here > currently. > > So rather something where I messed up during patch juggling. > > Perhaps still can be ยต-optimized with using lastindexof instead :) Given we talked about it and my brain has created an item, D23674 <https://phabricator.kde.org/D23674> is to get rid of it again :) REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D23625 To: kossebau, #kate, cullmann Cc: apol, cullmann, kwrite-devel, kde-frameworks-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns, demsking, sars, dhaumann
