meven added a comment.

  In D29397#663800 <https://phabricator.kde.org/D29397#663800>, @dfaure wrote:
  
  > Oh, I thought it was sent as an int. But 8 is 
QImage::Format_ARGB8565_Premultiplied. Did you mean 0x80?
  
  
  No I meant 8, since format is passed as quint8 (a single byte, 8-bit) and the 
format is forced to QImage::Format_ARGB32 "5", so this works.
  From kio-extra thumbnails :
  
    if( img.format() != QImage::Format_ARGB32 ) { // KIO::PreviewJob and this 
code below completely ignores colortable :-/,
        img = img.convertToFormat(QImage::Format_ARGB32); //  so make sure 
there is none
    }
    // Keep in sync with kio/src/previewjob.cpp

REPOSITORY
  R241 KIO

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

To: meven, dfaure, broulik, #frameworks
Cc: davidedmundson, kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, 
bruns

Reply via email to