astippich accepted this revision. astippich added a comment. This revision is now accepted and ready to land.
Just noticed, you never use the AppDataParser.name(). Is that intentional? Otherwise looks good, but you may want to wait for someone more experienced than me. REPOSITORY R286 KFileMetaData BRANCH addappimageextractor REVISION DETAIL https://phabricator.kde.org/D18450 To: kossebau, #baloo, astippich Cc: TheAssassin, astippich, broulik, kde-frameworks-devel, ashaposhnikov, michaelh, spoorun, ngraham, bruns, abrahams