Differential D16494

Take 1..n KMainWindows in kRestoreMainWindows
ClosedPublic
Actions

Authored by tobiasdeiminger on Oct 28 2018, 8:14 PM.

Details

Reviewers
apol
Commits
R263:726684c0eb19: Take 1..n KMainWindows in kRestoreMainWindows
Summary

We've got C++11 and can turn KMainWindow::kRestoreMainWindows into a variadic template to support more than 3 different toplevel widgets. Gives also minor code deduplication.

Test Plan

Compile and execute kmainwindowrestoretest.

Diff Detail

Repository
R263 KXmlGui
Branch
restore_more_windows
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 4284
Build 4302: arc lint + arc unit
Restricted Application added a project: Frameworks. · View Herald TranscriptOct 28 2018, 8:14 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
tobiasdeiminger requested review of this revision.Oct 28 2018, 8:14 PM
apol accepted this revision.Oct 29 2018, 12:41 AM
This revision is now accepted and ready to land.Oct 29 2018, 12:41 AM
Closed by commit R263:726684c0eb19: Take 1..n KMainWindows in kRestoreMainWindows (authored by tobiasdeiminger). · Explain WhyOct 29 2018, 8:41 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/kmainwindow.h (49 lines)

Diff 44383

View Options

src/kmainwindow.h

Loading...