Differential D16657

[kioslave/desktop] Remove more QDBusInterface
ClosedPublic
Actions

Authored by davidedmundson on Nov 4 2018, 9:27 AM.

Details

Reviewers
apol
Group Reviewers
Plasma
Commits
R120:c29d2528c184: [kioslave/desktop] Remove more QDBusInterface
Summary

No need to introspect the service and then call a method by a hardcooded
name.

Test Plan

Compiles

Diff Detail

Repository
R120 Plasma Workspace
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 4535
Build 4553: arc lint + arc unit
davidedmundson created this revision.Nov 4 2018, 9:27 AM
Restricted Application added a project: Plasma. · View Herald TranscriptNov 4 2018, 9:27 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Nov 4 2018, 9:27 AM
apol accepted this revision.Nov 5 2018, 12:20 AM
This revision is now accepted and ready to land.Nov 5 2018, 12:20 AM
Closed by commit R120:c29d2528c184: [kioslave/desktop] Remove more QDBusInterface (authored by davidedmundson). · Explain WhyNov 8 2018, 1:34 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 44820

View Options

kioslave/desktop/CMakeLists.txt

Loading...
View Options

kioslave/desktop/kio_desktop.cpp

Loading...