diff options
| author | Allan Sandfeld Jensen <allan.jensen@theqtcompany.com> | 2015-04-20 18:30:07 +0200 |
|---|---|---|
| committer | Allan Sandfeld Jensen <allan.jensen@theqtcompany.com> | 2015-04-24 09:03:48 +0000 |
| commit | 71203b1f0ec2da27d4cc03a82162eb7f643cc525 (patch) | |
| tree | 689734c34eb225b437bbee3524d74e494f3ed0ea /tests/auto/gui | |
| parent | 74f28ef1ca3be73dba4e8157729bab4cae40385d (diff) | |
Fix overflow in rgb30 gradient tests
The tests were trying to access one pixel outside of the image.
Change-Id: Ieabdefbbdb76bb736214a0495bec72c881b27d2c
Reviewed-by: Gunnar Sletta <gunnar@sletta.org>
Diffstat (limited to 'tests/auto/gui')
| -rw-r--r-- | tests/auto/gui/painting/qpainter/tst_qpainter.cpp | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/tests/auto/gui/painting/qpainter/tst_qpainter.cpp b/tests/auto/gui/painting/qpainter/tst_qpainter.cpp index 53e9ad6f20..e13a6b026d 100644 --- a/tests/auto/gui/painting/qpainter/tst_qpainter.cpp +++ b/tests/auto/gui/painting/qpainter/tst_qpainter.cpp @@ -3978,9 +3978,9 @@ void tst_QPainter::linearGradientRgb30() painter.fillRect(image.rect(), gradient); painter.end(); - for (int i = 0; i < 1000; ++i) { - QColor p1 = image.pixelColor(i, 0); - QColor p2 = image.pixelColor(i + 1, 0); + for (int i = 1; i < 1000; ++i) { + QColor p1 = image.pixelColor(i - 1, 0); + QColor p2 = image.pixelColor(i, 0); QVERIFY(p1 != p2); QVERIFY(qGray(p1.rgb()) >= qGray(p2.rgb())); } @@ -4005,9 +4005,9 @@ void tst_QPainter::radialGradientRgb30() painter.fillRect(image.rect(), gradient); painter.end(); - for (int i = 0; i < 1000; ++i) { - QColor p1 = image.pixelColor(i, 0); - QColor p2 = image.pixelColor(i + 1, 0); + for (int i = 1; i < 1000; ++i) { + QColor p1 = image.pixelColor(i - 1, 0); + QColor p2 = image.pixelColor(i, 0); QVERIFY(p1 != p2); QVERIFY(qGray(p1.rgb()) >= qGray(p2.rgb())); } |