D16789: Abstract OutputDevice initialization
Roman Gilg <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
romangg added a comment. In D16789#358969 <https://phabricator.kde.org/D16789#358969>, @davidedmundson wrote: > I'm not loving the name, the goal of this patchset is to move all of the wayland logic out of the platform. > > initWaylandOutput/initXdgOutput are now handled in setEnabled so can be private to AbstractOutput. > We should aim for the same with outputdevice. > > I'd rather this was just "AbstractOutput::init()" > > Code otherwise is fine Yea, makes sense. I'll take another look at it after this patch series has landed. REPOSITORY R108 KWin BRANCH abstractOutputDeviceInit REVISION DETAIL https://phabricator.kde.org/D16789 To: romangg, #kwin, davidedmundson Cc: davidedmundson, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart