Review Request 116991: kded: Use a consistent configuration file name

Alex Merry alex.merry at kde.org
Sun Mar 23 14:50:10 UTC 2014


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

(Updated March 23, 2014, 2:50 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and David Faure.


Repository: kded


Description
-------

Use a consistent configuration file name

Before, the initial configuration was explicitly drawn from "kdedrc",
while autoloading stuff was drawn from the application-default file,
which was "kded5rc".  This change makes it always use the
application-default file, which involves rearrangeing some of the code
to make sure the application instance always has its name set
consistently.

This also changes the configuration entry reading so that the defaults
are set in exactly one place.


Diffs
-----

  src/kded.cpp b409c69e101ef67558428c4b80f0e075e01599a3 

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


Testing
-------

Ran with some debug statements to print the value of CheckFileStamps both with and without --check.  Setting the value in ~/.config/kded5rc altered this printout.  This is also the file written to and read from for the module auto-loading stuff (checked using kded's D-Bus interface).


Thanks,

Alex Merry

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20140323/607f5a56/attachment.html>


More information about the Kde-frameworks-devel mailing list