davidedmundson updated this revision to Diff 79634.
davidedmundson added a comment.
and merge methods
REPOSITORY
R285 KCrash
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28336?vs=78636=79634
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D28336
AFFECTED FILES
This revision was automatically updated to reflect the committed changes.
Closed by commit R285:d8e41e64b7d0: Drop klauncher usage from KCrash (authored
by davidedmundson).
REPOSITORY
R285 KCrash
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D28336?vs=79634=79635
REVISION DETAIL
sitter accepted this revision.
sitter added inline comments.
This revision is now accepted and ready to land.
INLINE COMMENTS
> kcrash.cpp:627
>
> void KCrash::startProcess(int argc, const char *argv[], bool waitAndExit)
> {
This seems to serve no purpose anymore. startProcessInternal could
davidedmundson created this revision.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
davidedmundson requested review of this revision.
REVISION SUMMARY
As per other discussions, kdeinit has limited gains and is not widely
deployed in KDE application.