[tabswitcher] Fix beginRemoveRows call
ClosedPublic

Authored by davidedmundson on Oct 22 2018, 4:05 PM.

Details

Summary

To remove one row first and last should be the same index.

This hit an assert on newer Qt.

Test Plan

I hit this booting up with no args with kate
restoring to split views each with one document.

From the calling code it seems to be trying to just remove one row.

It now starts. Hitting control+n, control_w does add/remove tabs respectively

Diff Detail

Repository
R40 Kate
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
davidedmundson created this revision.Oct 22 2018, 4:05 PM
Restricted Application added a project: Kate. · View Herald TranscriptOct 22 2018, 4:05 PM
Restricted Application added a subscriber: kwrite-devel. · View Herald Transcript
davidedmundson requested review of this revision.Oct 22 2018, 4:05 PM
dhaumann accepted this revision.Oct 22 2018, 8:33 PM
dhaumann added a subscriber: dhaumann.

Thanks!

This revision is now accepted and ready to land.Oct 22 2018, 8:33 PM
This revision was automatically updated to reflect the committed changes.