ngraham added a comment.
Haha the "before" image is hilarious!
INLINE COMMENTS
> thumbnail.cpp:415
> + } else
> + if ((image.size().height() > imageTargetSize.height()) &&
> (imageTargetSize.height() != 0)) {
> + scaling = float(imageTargetSize.height()) /
> float(image.size().height());
Is this an `else if (...) {` or an `else { if (...) {`
REPOSITORY
R320 KIO Extras
REVISION DETAIL
https://phabricator.kde.org/D29806
To: meven, #frameworks, ngraham
Cc: kde-frameworks-devel, kfm-devel, waitquietly, azyx, nikolaik, pberestov,
iasensio, aprcela, fprice, LeGast00n, cblack, fbampaloukas, alexde, Codezela,
feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham,
andrebarros, bruns, emmanuelp, rdieter, mikesomov
