shubham added a comment.
@dhaumann I don't think using switch when there is only a single case good thought INLINE COMMENTS > sars wrote in kateviewinternal.cpp:2702 > does this even compile on OSX? Why did you change this? I can't tell, I don't have one. Btw Q_OS_OSX is deprecated. > sars wrote in kateviewinternal.cpp:2718 > Where did e->accept() / e->ignore() go? There is a reason they are there... By mistake i removed those after seeing their presence in switch, I will add those. 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