Fix default implementations of raster ops

In the list of default implementations, the raster ops added in
ae0ddb8c729c105a5b4f32a4f6765af8fe546333 were all offset by one
composition mode because of a duplicate entry in the array. The
effect would be, e.g. that using the NotDestination operator would
resolve to the Set operator instead.

Most users will probably not have experienced this since any of
the asm-based functions will be preferred.

[ChangeLog][Painting] Fixed some very rare cases of mismatched
raster modes in QPainter.

Change-Id: Ia242b54c78acbe1c89d9b4ecd10936564ec134b2
Task-number: QTBUG-41413
Reviewed-by: Andy Shaw <andy.shaw@digia.com>

1 files changed, 17 insertions, 0 deletions

diff --git a/tests/auto/gui/painting/qpainter/tst_qpainter.cpp b/tests/auto/gui/painting/qpainter/tst_qpainter.cpp
index 5af5b1a269..0675a6cf9d 100644
--- a/tests/auto/gui/painting/qpainter/tst_qpainter.cpp
+++ b/tests/auto/gui/painting/qpainter/tst_qpainter.cpp
@@ -296,6 +296,8 @@ private slots:
     void blendARGBonRGB_data();
     void blendARGBonRGB();
 
+    void RasterOp_NotDestination();
+
 private:
     void fillData();
     void setPenColor(QPainter& p);
@@ -4803,6 +4805,21 @@ void tst_QPainter::cosmeticStrokerClipping()
     QCOMPARE(old, image);
 }
 
+void tst_QPainter::RasterOp_NotDestination()
+{
+    QImage image(3, 3, QImage::Format_RGB32);
+    image.fill(Qt::red);
+
+    {
+        QPainter p(&image);
+        p.setCompositionMode(QPainter::RasterOp_NotDestination);
+        p.fillRect(image.rect(), Qt::black);
+    }
+
+    uint pixel = image.pixel(1, 1);
+    QCOMPARE(pixel, 0xff00ffff);
+}
+
 QTEST_MAIN(tst_QPainter)
 
 #include "tst_qpainter.moc"