croick marked an inline comment as done. croick added a comment.
In https://phabricator.kde.org/D7167#146752, @dhaumann wrote: > I think the patch makes sense, but I would prefer using a Qt function qPopulationCount to count the bits instead of doing own bitmask magic. Could you provide an updated patch? It's insignificantly slower now, but certainly easier to read. Thanks. REPOSITORY R39 KTextEditor REVISION DETAIL https://phabricator.kde.org/D7167 To: croick, #ktexteditor, dhaumann Cc: dhaumann, kwrite-devel, #frameworks