Review Request 127774: Load protocol info as a simple config.

David Edmundson david at davidedmundson.co.uk
Thu Apr 28 15:40:16 UTC 2016


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

(Updated April 28, 2016, 3:40 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Changes
-------

Submitted with commit 4a94cb0b6f76025b3d8cde92e7d0a26852842abf by David Edmundson to branch master.


Repository: kio


Description
-------

Protocol information is loaded from a full path on the system. It
doesn't make sense to also load either the user or system's kdeglobals.

This saves parsing kdeglobals ~40 times (see thread in /r/127765)


Diffs
-----

  src/core/kprotocolinfo.cpp 17f8e9d66bdc9ef257fda6490e9278ec6de79b5a 

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


Testing
-------

Dolphin still loads..


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160428/2b8a0e28/attachment.html>


More information about the Kde-frameworks-devel mailing list