aacid accepted this revision. aacid added a comment. This revision is now accepted and ready to land.
Good find :) To make the patch even a bit more awesome we should count the number of times openUrl fails and adapt desiredTab accordingly. I.e. if desiredTab is 4 but openinng url 1 fails we need to use 3 as desiredTab and not 4. But that's an already existing bug so if you don't feel like fixing it, it's fine, we cna fix it later :) REPOSITORY R223 Okular BRANCH avoid_crash_session_restore REVISION DETAIL https://phabricator.kde.org/D16457 To: tobiasdeiminger, aacid Cc: aacid, okular-devel, ngraham