davidedmundson added a comment.
Generally +1 INLINE COMMENTS > managedconfigmodule.cpp:43 > ManagedConfigModule *_q; > - QList<KCoreConfigSkeleton*> _skeletons; > + QList<QPointer<KCoreConfigSkeleton>> _skeletons; > }; Any reason for doing this approach rather than connecting to QObject:: destroyed and cleaning the list as we go? REPOSITORY R296 KDeclarative REVISION DETAIL https://phabricator.kde.org/D26046 To: crossi, #plasma, #frameworks, ervin, bport, davidedmundson, mart, meven Cc: meven, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
