[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    Re: Is this a bug in kdelibs4 ?
From:       David Faure <faure () kde ! org>
Date:       2006-06-23 14:03:42
Message-ID: 200606231603.43131.faure () kde ! org
[Download RAW message or body]

On Friday 23 June 2006 14:12, Tomas Mecir wrote:
> Hi everyone,
> 
> I looked through the code of the KDialog in kdelibs4-snapshot, in an
> attempt to locate my crash, and I succeeded. So I'd like to ask - is
> the following to be considered a bug in kdelibs4 ? Because to me, it
> seems that way.
> 
> Constructor of KDialog sets default buttons to Ok | Cancel.
> This constructor also sets default button to Ok (setDefaultButton)- as
> a pointer to the Ok button widget.
> When you call setButtons to change buttons (or just to ensure that
> you'll really have Ok|Cancal, as the Sort dialog is doing), the old
> buttons are deleted and new ones created, BUT the default button
> remains the same pointer.
> As a result, unless you also call setDefaultButton in your code, you
> will get a crash upon trying to display the widget (QDialog tries to
> call setFocus on the default widget, but the pointer points to deleted
> memory).
> 
> Soo ... is this a bug in KDialog, which should clear the default
> button in the setButtons call ? And if so, how to best report ?
> kdelibs-core-devel or bugzilla ?

Tobias Koenig has investigated this bug already, and I think that kdelibs
has a workaround already.

-- 
David Faure, faure@kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).

_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]