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

(Updated April 29, 2015, 7:48 a.m.)


Status
------

This change has been marked as submitted.


Review request for Telepathy and Martin Klapetek.


Changes
-------

Submitted with commit 4d9dfcc1d47b5b99e822148650b36dffbc10da26 by David Rosca 
to branch Applications/15.04.


Repository: ktp-accounts-kcm


Description
-------

Connect QDialogButtonBox rejected signal to QDialog reject slot.

This fixes crash when closing add new account dialog with close button (or 
Alt+F4).


Diffs
-----

  plugins/kaccounts/kaccounts-ui-provider.cpp a7569cf 

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


Testing
-------

No longer crashes when closing the dialog with close button and then trying to 
open new dialog - this can be triggered just by opening Gadu-Gadu new acc 
dialog -> closing it with Alt+F4 and then opening it again.

However, it does not fix a crash when clicking on "Create" entry in accounts 
list view -> then switching to some configured account -> switch back to 
"Create" and try to open few new account dialogs. The crash here seems to be 
related to KWidgetItemDelegate (particularly to createItemWidgets), but I'm not 
really sure what might be wrong as widgets in createItemWidgets are created 
correctly without parent.

This is a warning message when closing the dialog:

    KWidgetItemDelegateEventListener::eventFilter-e : User of 
KWidgetItemDelegate should not delete widgets created by createItemWidgets!


Thanks,

David Rosca

_______________________________________________
KDE-Telepathy mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-telepathy

Reply via email to