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

Looks good to me, but maybe let dfaure have a second look

- Kevin Krammer


On April 14th, 2014, 11:48 a.m. UTC, David Jarvie wrote:

Review request for kdelibs.
By David Jarvie.

Updated April 14, 2014, 11:48 a.m.

Repository: kdelibs

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.

Testing 

Tested this scenario, and it now returns the error string.

Diffs

  • kio/kio/kdbusservicestarter.cpp (90624fb)

View Diff