mwolff accepted this revision. mwolff added a comment. This revision is now accepted and ready to land.
lgtm, despite the `delete this`: https://isocpp.org/wiki/faq/freestore-mgmt#delete-this, esp. if you have proper unit tests and ran it through valgrind REPOSITORY R241 KIO BRANCH master REVISION DETAIL https://phabricator.kde.org/D7529 To: dfaure, mwolff Cc: #frameworks