bruns accepted this revision.
bruns added a comment.
This revision is now accepted and ready to land.


  Thanks for the explanation. Looks good then, just update the summary.
  
  You could add another column to the tests, "hasFullImplementation", and do an 
Q_EXPECT_FAIL if not set. But thats for another patch ...

REPOSITORY
  R286 KFileMetaData

BRANCH
  refactor_taglib_extractor_tests

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

To: astippich, bruns, svuorela
Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, 
ngraham, bruns, abrahams

Reply via email to