Review Request: Add option to KGlobalConfig::activate() to skip connection to dbus

[David Faure]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/5072/#review7145
-----------------------------------------------------------

Ship it!


Looks good, thanks. Just missing documentation for the enum values, and it's good to go ;)

- David


On 2010-08-19 15:48:48, George Kiagiadakis wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/5072/
> -----------------------------------------------------------
> 
> (Updated 2010-08-19 15:48:48)
> 
> 
> Review request for kdelibs, David Faure and Oswald Buddenhagen.
> 
> 
> Summary
> -------
> 
> This patch modifies KGlobalConfig so that it is possible for applications to tell it not to connect to dbus and listen for configuration changes from other applications. This is useful for kdm, because if QDBusConnection::sessionBus() is called while there is no session bus running, a new session bus is automatically created, which makes no sense for kdm.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdelibs/kdeui/kernel/kglobalsettings.h 1165324 
>   trunk/KDE/kdelibs/kdeui/kernel/kglobalsettings.cpp 1165324 
> 
> Diff: http://reviewboard.kde.org/r/5072/diff
> 
> 
> Testing
> -------
> 
> Tested with kdm after modifying its call to activate() to use the DontListenForChanges flag. kdm no longer starts a dbus session.
> 
> 
> Thanks,
> 
> George
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100820/a5d38cd7/attachment.htm>