dhaumann added a comment.
Herald edited subscribers, added: kde-frameworks-devel, kwrite-devel; removed: 
Frameworks.


  Curent state: This patch is only almost good to go in... :-P

INLINE COMMENTS

> cullmann wrote in kateview.cpp:3595
> I think the old code tried to ensure we only add word-boundary if we still 
> can find the string then. e.g. if the string that you highlight contains 
> spaces or stuff like that at the borders, you don't add \b.

This part is still not done. Besides that, this patch should be fine.

REPOSITORY
  R39 KTextEditor

REVISION DETAIL
  https://phabricator.kde.org/D9175

To: dhaumann, cullmann, kfunk, mwolff
Cc: kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, 
demsking, cullmann, sars, dhaumann, #frameworks

Reply via email to