Review Request: Do not leak kdeuiwidgetsproxystyle

Boudewijn Rempt boud at valdyas.org
Fri Oct 8 11:07:23 BST 2010


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

Review request for kdelibs.


Summary
-------

For instance in KLineEdit::init(), we have a line like:

d->style = new KLineEditStyle(this);

This gives the impression that KLineEditStyle has QObject-based parent-child relation and will be deleted automatically. That wasn't true, because QStyle while being a QObject doesn't have a QObject* parent paramater in its constructor. So this patch sets the parent explicitly in KdeUiProxyStyle 


Diffs
-----

  trunk/KDE/kdelibs/kdeui/widgets/kdeuiwidgetsproxystyle.cpp 1182756 

Diff: http://svn.reviewboard.kde.org/r/5549/diff


Testing
-------


Thanks,

Boudewijn

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


More information about the kde-core-devel mailing list