D16494: Take 1..n KMainWindows in kRestoreMainWindows

2018-10-29 Thread Tobias Deiminger
This revision was automatically updated to reflect the committed changes.
Closed by commit R263:726684c0eb19: Take 1..n KMainWindows in 
kRestoreMainWindows (authored by tobiasdeiminger).

REPOSITORY
  R263 KXmlGui

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D16494?vs=44383=44464

REVISION DETAIL
  https://phabricator.kde.org/D16494

AFFECTED FILES
  src/kmainwindow.h

To: tobiasdeiminger, apol
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D16494: Take 1..n KMainWindows in kRestoreMainWindows

2018-10-28 Thread Aleix Pol Gonzalez
apol accepted this revision.
This revision is now accepted and ready to land.

REPOSITORY
  R263 KXmlGui

BRANCH
  restore_more_windows

REVISION DETAIL
  https://phabricator.kde.org/D16494

To: tobiasdeiminger, apol
Cc: kde-frameworks-devel, michaelh, ngraham, bruns


D16494: Take 1..n KMainWindows in kRestoreMainWindows

2018-10-28 Thread Tobias Deiminger
tobiasdeiminger created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
tobiasdeiminger requested review of this revision.

REVISION 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.

REPOSITORY
  R263 KXmlGui

BRANCH
  restore_more_windows

REVISION DETAIL
  https://phabricator.kde.org/D16494

AFFECTED FILES
  src/kmainwindow.h

To: tobiasdeiminger
Cc: kde-frameworks-devel, michaelh, ngraham, bruns