Review Request 116462: Keep KSharedConfigPtr("kdeglobals") around in KHintSettings.

David Faure faure at kde.org
Thu Feb 27 21:59:27 UTC 2014


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

Review request for KDE Frameworks and Aleix Pol Gonzalez.


Repository: frameworkintegration


Description
-------

Keep KSharedConfigPtr("kdeglobals") around in KHintSettings.

This makes KFontSettingsData end up using the same instance, rather than
each of these two classes parsing kdeglobals on startup, in turn.

Had to fix the unittest again, it doesn't use QStandardPaths' testMode,
so it must ensure to set its special environment before the platformtheme
is created.

strace -e open kate |& grep -v NOENT | grep kdeglobals | wc -l
  went from 5 to 4.


Diffs
-----

  src/platformtheme/khintssettings.cpp c4de4badc6187d841af36e29e083ffd2ca475d82 
  src/platformtheme/khintssettings.h 8f353837884590bc6a8409df7273435824273f49 
  src/platformtheme/kfontsettingsdata.cpp 62990ce45c1a175c3b57710c8a38268d08908733 
  autotests/kfontsettingsdata_unittest.cpp 4ee33fea72905128112226248667499489b1c692 

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


Testing
-------

as described: unittests in frameworkintegration + running kate.


Thanks,

David Faure

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140227/381d0cb3/attachment.html>


More information about the Kde-frameworks-devel mailing list