D23939: [KService] add workingDirectory(), deprecate path().
David Faure
noreply at phabricator.kde.org
Sat Sep 14 12:41:37 BST 2019
dfaure created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
dfaure requested review of this revision.
REVISION SUMMARY
Reading code like service->path() is confusing, because it looks like
the path to the .desktop file (which is in fact entryPath(), most
likely because path() was already taken).
The key in the desktop file is called Path (and that won't change),
but in the C++ API, workingDirectory() is much clearer.
TEST PLAN
Builds
REPOSITORY
R309 KService
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D23939
AFFECTED FILES
src/services/kservice.cpp
src/services/kservice.h
src/services/kservice_p.h
To: dfaure
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190914/bda33568/attachment.html>
More information about the Kde-frameworks-devel
mailing list