D18788: Refactor KateViewInternal::mouseDoubleClickEvent(QMouseEvent *e)

[Dominik Haumann]

dhaumann added a comment.


  I am with Kåre here: this change does not fix any issue nor does it improve the current state significantly. In fact, there is a risk of introducing a regression by removing the accept/reject calls.
  
  I don't think this should go in as is.
  
  Shubham, on bugs.kde.org we have more than 300 open bugs. Maybe you can find something there? Would be really nice :)

REPOSITORY
  R39 KTextEditor

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

To: shubham, cullmann
Cc: dhaumann, sars, kwrite-devel, kde-frameworks-devel, hase, michaelh, ngraham, bruns, demsking, cullmann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190206/c48a727b/attachment.html>