[kcms/ksmserver] Port to new kworkspace API
ClosedPublic

Authored by davidedmundson on Jun 12 2019, 10:45 AM.

Details

Summary

requestShutDown is replaced

Test Plan

Bodged the code so that it made my system think it had firmware support
Clicked checkbox, pressed button. Got my shutdown prompt

Diff Detail

Repository
R119 Plasma Desktop
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 12896
Build 12914: arc lint + arc unit
Restricted Application added a project: Plasma. · View Herald TranscriptJun 12 2019, 10:45 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Jun 12 2019, 10:45 AM

update enum -> enum class

broulik accepted this revision.Jun 21 2019, 4:36 PM
broulik added a subscriber: broulik.
broulik added inline comments.
kcms/ksmserver/kcmsmserver.cpp
86

Coding style.

88

Is this safe, or should it do a deleteLater()?

This revision is now accepted and ready to land.Jun 21 2019, 4:36 PM
This revision was automatically updated to reflect the committed changes.