bruns added a comment.

  Untested, but looks good to me otherwise.

INLINE COMMENTS

> taglibextractortest.cpp:42
>  
> +const QStringList TagLibExtractorTest::propertyEnumNames(const 
> QList<KFileMetaData::Property::Property>& keys) const
> +{

Nitpick - needs some indentation

REPOSITORY
  R286 KFileMetaData

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

To: michaelh, mgallien, #baloo, #frameworks
Cc: bruns, astippich, ashaposhnikov, michaelh, spoorun, ngraham, alexeymin

Reply via email to