Removed unncessary switch and replaced it with simple if, more efficient
Details
Details
Diff Detail
Diff Detail
- Repository
- R39 KTextEditor
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
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 :)
src/view/kateviewinternal.cpp | ||
---|---|---|
2718 | Btw no need to explicitly accept() the event, because isAccepted() function returns true by default. But surely, event should be explicitly ignored. |
Comment Actions
I think it's ok, although it is arguably whether this improves anything. Still, let's move on.