Review Request 128902: [Kauth] - Add timeout property

stephane cachemaille stephane at scachemaille.ch
Tue Nov 8 01:12:36 UTC 2016


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

(Updated Nov. 8, 2016, 2:12 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Changes
-------

Submitted with commit 99938b0980131202c5d3e2fb1845106c147917d1 by Aleix Pol on behalf of stephane cachemaille to branch master.


Bugs: 363200
    https://bugs.kde.org/show_bug.cgi?id=363200


Repository: kauth


Description
-------

Add timeout property to Kauth::Action class to be able to change the timeout in QDBusConnection::asyncCall


Diffs
-----

  src/HelperProxy.h 96f4cf1 
  src/backends/dbus/DBusHelperProxy.h 8f4254c 
  src/backends/dbus/DBusHelperProxy.cpp d2620ac 
  src/backends/fakehelper/FakeHelperProxy.h d0fb3f5 
  src/backends/fakehelper/FakeHelperProxy.cpp a044ac6 
  src/kauthaction.h 4e33366 
  src/kauthaction.cpp 34f559d 
  src/kauthexecutejob.cpp 8b6e177 

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


Testing
-------

tested with and without using the new proporty in a kcm module where the action takes more than the default 25s


Thanks,

stephane cachemaille

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161108/e993241b/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list