Differential D22387

Allow KAutostart to be constructed using an absolute path
ClosedPublic
Actions

Authored by apol on Jul 10 2019, 6:44 PM.

Details

Reviewers
davidedmundson
Group Reviewers
Frameworks
Commits
R309:93107a9532e9: Allow KAutostart to be constructed using an absolute path
Summary

If we already have found the service we need to autostart, it's a waste
to strip the full path to have it located again.

Test Plan

tests still pass, works with my plasma-workspace patch

Diff Detail

Repository
R309 KService
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
apol created this revision.Jul 10 2019, 6:44 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJul 10 2019, 6:44 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
apol requested review of this revision.Jul 10 2019, 6:44 PM
apol retitled this revision from Allow the instance to be constructed using an absolute path to Allow KAutostart to be constructed using an absolute path.Jul 10 2019, 7:03 PM
davidedmundson accepted this revision.Jul 10 2019, 9:36 PM
This revision is now accepted and ready to land.Jul 10 2019, 9:36 PM
Closed by commit R309:93107a9532e9: Allow KAutostart to be constructed using an absolute path (authored by apol). · Explain WhyJul 11 2019, 2:36 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/services/kautostart.h (3 lines)
Msrc/services/kautostart.cpp (20 lines)

Diff 61559

View Options

src/services/kautostart.h

Loading...
View Options

src/services/kautostart.cpp

Loading...