and deprecate the QWidget * overload, so that we can get rid of the
QtWidgets dependency one day.
Details
Details
Builds, new test passes
Diff Detail
Diff Detail
- Repository
- R278 KWindowSystem
- Branch
- 2019_setNewStartupId
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 14978 Build 14996: arc lint + arc unit
Comment Actions
Add test for QWidget overload and fix crash
I found out a comment in KWindowSystem::setMainWindow which said this was necessary
(urgh, bad API), so doing the same here, after the unittest showed that windowHandle()
is indeed nullptr otherwise.
If this commit is OK, then KWindowSystem::setMainWindow() is the (only) other one
which will need similar treatment.