ubuntu-touch-coreapps-reviewers team mailing list archive

[Roman Shchekin <mrqtros@xxxxxxxxx>]

Roman Shchekin has proposed merging lp:~mrqtros/ubuntu-docviewer-app/lo-tiled-rendering-bgr-to-rgb into lp:~ubuntu-docviewer-dev/ubuntu-docviewer-app/lo-tiled-rendering.

Commit message:
Seems that LoDocument uses BGR rather than RGB. Fixed.

Requested reviews:
  Ubuntu Document Viewer Developers (ubuntu-docviewer-dev)

For more details, see:
https://code.launchpad.net/~mrqtros/ubuntu-docviewer-app/lo-tiled-rendering-bgr-to-rgb/+merge/264473

Seems that LoDocument uses BGR rather than RGB. Fixed.
Before:
https://yadi.sk/i/MlMK6xtPhoVxT
After:
https://yadi.sk/i/QEAI8cY1hoVxV
-- 
Your team Ubuntu Document Viewer Developers is requested to review the proposed merge of lp:~mrqtros/ubuntu-docviewer-app/lo-tiled-rendering-bgr-to-rgb into lp:~ubuntu-docviewer-dev/ubuntu-docviewer-app/lo-tiled-rendering.

=== modified file 'src/plugin/libreofficetoolkit-qml-plugin/lodocument.cpp'
--- src/plugin/libreofficetoolkit-qml-plugin/lodocument.cpp	2015-07-04 16:00:33 +0000
+++ src/plugin/libreofficetoolkit-qml-plugin/lodocument.cpp	2015-07-11 10:50:22 +0000
@@ -113,7 +113,7 @@
                           Twips::convertPixelsToTwips(tileSize.width()),
                           Twips::convertPixelsToTwips(tileSize.height()));
 
-    return result;
+    return result.rgbSwapped();
 }
 
 LODocument::~LODocument()