D16064: Fix memleaks in unittests

Bernd Buschinski noreply at phabricator.kde.org
Tue Oct 9 11:08:58 BST 2018


buschinski added inline comments.

INLINE COMMENTS

> kfunk wrote in test_projectmodel.cpp:325
> Here and below (where you use `delete ...`): Try to use `QScopedPointer` instead of raw pointers -- that way you don't need to cleanup yourself at the end of the block?
> 
> And it also cleans up memory even if we bail out at any of the `QCOMPARE` or `QVERIFY` stmts.

Good hint! Will do :)

REPOSITORY
  R32 KDevelop

REVISION DETAIL
  https://phabricator.kde.org/D16064

To: buschinski, #kdevelop, kfunk
Cc: kfunk, kdevelop-devel, glebaccon, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20181009/1bb1e50d/attachment.html>


More information about the KDevelop-devel mailing list