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

Reply via email to