Review Request 129042: Replace all blocking calls when initializing properties by just one blocking call to get them all

Jan Grulich jgrulich at redhat.com
Fri Sep 30 08:15:54 UTC 2016


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

(Updated Sept. 30, 2016, 11:15 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks, Network Management, Aleix Pol Gonzalez, and Lamarque Souza.


Changes
-------

Submitted with commit 00acf17a2071d86e218d3d49a0e53157eaa265dd by Jan Grulich to branch master.


Repository: networkmanager-qt


Description
-------

Uses "GetAll" method from o.f.DBus.Properties interface to get all properties at once, instead of initializing them one by one, which should significantly reduce number of blocking calls. Initialy I wanted to make this call asynchronous as well, but delayed initialization causes problems across the library so this is the easiest way how to do this.


Diffs
-----

  src/accesspoint.cpp 60a284b 
  src/activeconnection.cpp 1b66fc5 
  src/activeconnection_p.h 60697e6 
  src/adsldevice.cpp 8f9bbff 
  src/bluetoothdevice.cpp eb839e0 
  src/bonddevice.cpp 6d1899f 
  src/bridgedevice.cpp 3c347af 
  src/device.cpp 57538eb 
  src/genericdevice.cpp 8f26617 
  src/gredevice.cpp c1a59c5 
  src/infinibanddevice.cpp 9bfa76f 
  src/macvlandevice.cpp 9b3c75f 
  src/manager.cpp d2f706e 
  src/manager_p.h 50c3c34 
  src/modemdevice.cpp 4bb4a23 
  src/olpcmeshdevice.cpp 6f1212d 
  src/settings.cpp 90e7267 
  src/teamdevice.cpp d07b145 
  src/tundevice.cpp 2a73012 
  src/vethdevice.cpp 77b48b4 
  src/vlandevice.cpp fdfa297 
  src/vpnconnection.cpp 5dee6fc 
  src/vpnconnection_p.h a8195ad 
  src/wimaxdevice.cpp 6118f2f 
  src/wireddevice.cpp 66bb649 
  src/wirelessdevice.cpp 3f2972b 

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


Testing
-------


Thanks,

Jan Grulich

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160930/4d6db762/attachment.html>


More information about the Kde-frameworks-devel mailing list