Review Request: Fix Warning messages when closing About popup

Commit Hook null at kde.org
Mon Aug 6 20:07:48 BST 2012


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


This review has been submitted with commit 9aaa24814030041f51405182f03c9760e04bc76f by Albert Astals Cid to branch KDE/4.9.

- Commit Hook


On July 19, 2012, 10:23 p.m., Albert Astals Cid wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105627/
> -----------------------------------------------------------
> 
> (Updated July 19, 2012, 10:23 p.m.)
> 
> 
> Review request for kdelibs, Frederik Gladhorn and David Faure.
> 
> 
> Description
> -------
> 
> There is a too agressive check in KWidgetItemDelegateEventListener that complains about
> "User of KWidgetItemDelegate should not delete widgets created by createItemWidgets!"
> when closing an about dialog.
> 
> I tried making it less agressive but since i could not, i'm just deleting things in a different order in the destruction of KAboutApplicationDialog so that the warning is not triggered and our users get angry
> 
> 
> This addresses bug 301628.
>     http://bugs.kde.org/show_bug.cgi?id=301628
> 
> 
> Diffs
> -----
> 
>   kdeui/dialogs/kaboutapplicationdialog.cpp 2c98f74 
> 
> Diff: http://git.reviewboard.kde.org/r/105627/diff/
> 
> 
> Testing
> -------
> 
> Dialog still works, valgrind doesn't complain, the warning is gone
> 
> 
> Thanks,
> 
> Albert Astals Cid
> 
>

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


More information about the kde-core-devel mailing list