This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118545/

On June 5th, 2014, 12:58 p.m. CEST, David Faure wrote:

Correct, video/* isn't supported. In any case, how can you be sure that this application will support *ALL* video codecs?

On June 5th, 2014, 1:05 p.m. CEST, Hrvoje Senjan wrote:

i cannot - this was used in 4.x version, i've added a few specific ones validated they work. will adjust the review, and remove the /* completely...

On June 5th, 2014, 1:21 p.m. CEST, David Faure wrote:

Unless I'm wrong, video/* didn't work in kdelibs4 either.

(IIRC it worked for ServiceMenus, but not for application desktop files, which have to follow the Desktop Entry Spec).
maybe it *shouldn't* work with 4.x - but i do have thumbs in 4.x dolphin with such a desktop file ;-)
but then http://api.kde.org/frameworks-api/frameworks5-apidocs/kio/html/classThumbCreator.html has it still wrong (that's why i havent removed video/* completely):
"Note that you can supply a comma-separated list of mimetypes to the MimeTypes entry, naming all mimetypes your ThumbCreator supports. You can also use simple wildcards, like "text/*"."

- Hrvoje


On June 5th, 2014, 12:47 p.m. CEST, Hrvoje Senjan wrote:

Review request for KDE Multimedia and David Faure.
By Hrvoje Senjan.

Updated June 5, 2014, 12:47 p.m.

Repository: ffmpegthumbs

Description

simple port to KF5.
added Alex if he maybe has an idea why video/* doesn't seem to work.

Testing

Builds; thumbnails appear in Dolphin.

Diffs

  • ffmpegthumbs.desktop (1e2a95a)
  • ffmpegthumbnailer/imagewriter.cpp (7dae371)
  • ffmpegthumbnailer/moviedecoder.cpp (d63cd4b)
  • ffmpegthumbnailer.cpp (03a40c3)
  • CMakeLists.txt (dd0974f)

View Diff