Review Request 129923: [ffmpegthumbs] Remove the 1px border from thumbnails
Diego Soenens
hyperz.2007 at gmail.com
Mon Mar 27 23:50:16 BST 2017
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129923/
-----------------------------------------------------------
(Updated March 27, 2017, 10:50 p.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Multimedia.
Changes
-------
Submitted with commit bb2944b120ebf618a6e2fc5bec448127de441b04 by Albert Astals Cid on behalf of Diego Soenens to branch Applications/17.04.
Repository: ffmpegthumbs
Description
-------
This is part of https://git.reviewboard.kde.org/r/129921/.
Returning DrawFrame in flags results in a 1px black and white border getting added to the thumbnail while returning None does not.
There is no reason why it should be adding a border.
Especially since this thumbnail creator generates images that could (and should IMO) be treated the same as thumbnails from, say, image files like wallpapers etc.
Diffs
-----
ffmpegthumbnailer.cpp e8604e417979fecf9af5edda67cf8f1dbfc0f13c
Diff: https://git.reviewboard.kde.org/r/129923/diff/
Testing
-------
Thanks,
Diego Soenens
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20170327/572fc361/attachment.htm>
More information about the kde-multimedia
mailing list