kdialog/knotify and URLs

[Kevin Krammer]

On Monday 14 November 2005 23:33, Thomas Braxton wrote:
> On Monday 14 November 2005 03:03 pm, Kevin Krammer wrote:
> >+        int messageBoxOptions = KMessageBox::Notify;
> >+        KConfig* config = kapp->config();
> >+        if ( config ) {
> >+            KConfigGroupSaver groupSaver( config, "General" );
> >+            if ( config->readBoolEntry( "AllowLinks", false ) )
> >+                messageBoxOptions |= KMessageBox::AllowLink;
> >+        }
>
> You might want to change to KConfigGroup, because KConfigGroupSaver is
> going away for KDE4, this patch would be forward compatible. If you are
> changing 3.5 to contain this functionality, 4.0 would be expected to have
> it.

Good hint, thanks!

New patch attached

Cheers,
Kevin

-- 
Kevin Krammer <kevin.krammer at gmx.at>
Qt/KDE Developer, Debian User
Moderator: www.mrunix.de (German), www.qtforum.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdialog.diff
Type: text/x-diff
Size: 1289 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20051115/b82a5eb7/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20051115/b82a5eb7/attachment.sig>