-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117604/
-----------------------------------------------------------

(Updated April 22, 2014, 10:03 p.m.)


Review request for KDE Frameworks and David Faure.


Changes
-------

Fixed warning text.


Repository: kservice


Description
-------

When KDBusServiceStarter::findServiceFor() fails to start the requested service 
after it is found to not be running, it does not return the error string. This 
patch fixes that and makes it behave as in the apidox.

This is a copy of https://git.reviewboard.kde.org/r/116951/ for KDE 4, except 
that the qDebug line has been uncommented on the grounds that it reports an 
error.


Diffs (updated)
-----

  src/plugin/kdbusservicestarter.cpp 6f011bd 

Diff: https://git.reviewboard.kde.org/r/117604/diff/


Testing
-------

Tested for review 116951.


Thanks,

David Jarvie

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to