diff options
| author | Noel <noel.grandin@collabora.co.uk> | 2021-01-18 10:02:20 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-01-18 11:44:00 +0100 |
| commit | 1e9255303db01786f403253a1487cee0af56525d (patch) | |
| tree | 346f00391833e538b00dd015aa8db5d8a57c1a64 /sw | |
| parent | fea7b531a4c9e8326a644cccc26325b16585675a (diff) | |
make internals of ::Color private
was made public in
commit d487d6e082bc7ce652217578ffd37397a59cc3ca
rework Color to have R,G,B,A public variables
but it's best to keep them private since we're changing the internal
representation - and there is surprisingly little that depends
on accessing the internals anyhow.
Change-Id: I234d80fb33105f7bf5b865fb7c8194d893b09878
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/109508
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sw')
| -rw-r--r-- | sw/qa/extras/tiledrendering/tiledrendering.cxx | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sw/qa/extras/tiledrendering/tiledrendering.cxx b/sw/qa/extras/tiledrendering/tiledrendering.cxx index 10d54fc91e2e..28edd8c5fb55 100644 --- a/sw/qa/extras/tiledrendering/tiledrendering.cxx +++ b/sw/qa/extras/tiledrendering/tiledrendering.cxx @@ -2419,9 +2419,9 @@ void SwTiledRenderingTest::testSemiTransparent() // Without the accompanying fix in place, this test would have failed with 'Expected greater or // equal than: 190; Actual: 159'. This means the semi-transparent gray rectangle was darker than // expected, as it was painted twice. - CPPUNIT_ASSERT_GREATEREQUAL(190, static_cast<int>(aColor.R)); - CPPUNIT_ASSERT_GREATEREQUAL(190, static_cast<int>(aColor.G)); - CPPUNIT_ASSERT_GREATEREQUAL(190, static_cast<int>(aColor.B)); + CPPUNIT_ASSERT_GREATEREQUAL(190, static_cast<int>(aColor.GetRed())); + CPPUNIT_ASSERT_GREATEREQUAL(190, static_cast<int>(aColor.GetGreen())); + CPPUNIT_ASSERT_GREATEREQUAL(190, static_cast<int>(aColor.GetBlue())); } void SwTiledRenderingTest::testHighlightNumbering() @@ -2543,7 +2543,7 @@ void SwTiledRenderingTest::testClipText() for (int i = 0; i < 150; i++) { Color aTopTextColor(pAccess->GetPixel(98, 98 + i)); - if (aTopTextColor.R < 255) + if (aTopTextColor.GetRed() < 255) { bClipTop = false; break; |