This revision was automatically updated to reflect the committed changes.
Closed by commit R119:01095327f014: Understand KAStats-style applications:
launcher URLs. (authored by hein).
REPOSITORY
R119 Plasma Desktop
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7562?vs=18875&id=18876
broulik accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R119 Plasma Desktop
BRANCH
understandApplicationsURLs
REVISION DETAIL
https://phabricator.kde.org/D7562
To: hein, #plasma, broulik, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai,
hein updated this revision to Diff 18875.
hein added a comment.
Cleanup repeated code.
REPOSITORY
R119 Plasma Desktop
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D7562?vs=18819&id=18875
BRANCH
understandApplicationsURLs
REVISION DETAIL
https://phabricator.kde.org/D7562
AF
hein added a comment.
Yeah, probably a good idea (the code it replaces was also copy-pasted three
times though :).
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D7562
To: hein, #plasma, broulik, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai
broulik added a comment.
This seems to have been copy-pasted three times - can we put that into a
separate method?
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D7562
To: hein, #plasma, broulik, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai
hein created this revision.
Restricted Application added a project: Plasma.
REVISION SUMMARY
See https://phabricator.kde.org/D7561.
REPOSITORY
R119 Plasma Desktop
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D7562
AFFECTED FILES
applets/taskmanager/plugin/backend.cpp
To
