bruns added a comment.
..., otherwise, LGTM INLINE COMMENTS > ffmpegextractortest.cpp:53 > > -void ffmpegExtractorTest::videoTest() > +//test only for testing intrinsic video properties > +void ffmpegExtractorTest::testVideoProperties() a little bit to much "test" here ;) and missing space REPOSITORY R286 KFileMetaData REVISION DETAIL https://phabricator.kde.org/D20020 To: astippich, ngraham, bruns Cc: kde-frameworks-devel, #baloo, gennad, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams