summaryrefslogtreecommitdiff
path: root/basegfx
diff options
context:
space:
mode:
authorMiklos Vajna <vmiklos@collabora.com>2019-05-14 16:44:17 +0200
committerMiklos Vajna <vmiklos@collabora.com>2019-05-14 19:47:32 +0200
commit35ab83872f184c22c5f52f43ee38d3669340b192 (patch)
treebd9c502a69b07409f130837ad06d74377afe15df /basegfx
parent4cac171dd6c82f55624ba6e7185d69bcb5507cb1 (diff)
Revert "drawinglayer: avoid AA for hairline polygons built from ...
.. hori/vert lines only" This reverts commit f8b4d371eddd27594d549fb00294c01229a9bd24. Tomaz considers this ugly and it's no longer needed since commit 93abdf39b01bb7b404dc09ef37369a4350fb0d10 (sw lok: assume no windows in SwLayoutFrame::PaintSwFrame(), 2019-05-14). Conflicts: basegfx/test/B2DPolygonTest.cxx Change-Id: Ia9b29921ff3e5d82085e1abf9f39c172357a5e13 Reviewed-on: https://gerrit.libreoffice.org/72297 Reviewed-by: Miklos Vajna <vmiklos@collabora.com> Tested-by: Jenkins
Diffstat (limited to 'basegfx')
-rw-r--r--basegfx/source/polygon/b2dpolypolygontools.cxx18
-rw-r--r--basegfx/test/B2DPolygonTest.cxx19
2 files changed, 0 insertions, 37 deletions
diff --git a/basegfx/source/polygon/b2dpolypolygontools.cxx b/basegfx/source/polygon/b2dpolypolygontools.cxx
index d537531a4600..062e37da9dae 100644
--- a/basegfx/source/polygon/b2dpolypolygontools.cxx
+++ b/basegfx/source/polygon/b2dpolypolygontools.cxx
@@ -416,24 +416,6 @@ namespace basegfx
return aRetval;
}
- bool containsOnlyHorizontalOrVerticalLines(const basegfx::B2DPolygon& rPolygon)
- {
- if (rPolygon.count() <= 1)
- return false;
-
- for (size_t i = 1; i < rPolygon.count(); ++i)
- {
- const basegfx::B2DPoint& rPrevPoint = rPolygon.getB2DPoint(i - 1);
- const basegfx::B2DPoint& rPoint = rPolygon.getB2DPoint(i);
- if (rPrevPoint.getX() == rPoint.getX() || rPrevPoint.getY() == rPoint.getY())
- continue;
-
- return false;
- }
-
- return true;
- }
-
B2DPolyPolygon createSevenSegmentPolyPolygon(sal_Char nNumber, bool bLitSegments)
{
// config here
diff --git a/basegfx/test/B2DPolygonTest.cxx b/basegfx/test/B2DPolygonTest.cxx
index 46458574a270..0bb3f15dfdbe 100644
--- a/basegfx/test/B2DPolygonTest.cxx
+++ b/basegfx/test/B2DPolygonTest.cxx
@@ -22,7 +22,6 @@
#include <cppunit/extensions/HelperMacros.h>
#include <basegfx/polygon/b2dpolygon.hxx>
-#include <basegfx/polygon/b2dpolygontools.hxx>
#include <basegfx/point/b2dpoint.hxx>
namespace basegfx
@@ -79,30 +78,12 @@ public:
aPoly.getB2DPoint(1));
}
- void testContainsOnlyHorizontalOrVerticalLines()
- {
- // First line is horizontal, second is vertical.
- B2DPolygon aPoly;
- aPoly.append(B2DPoint(0, 1));
- aPoly.append(B2DPoint(1, 1));
- aPoly.append(B2DPoint(1, 0));
- CPPUNIT_ASSERT(utils::containsOnlyHorizontalOrVerticalLines(aPoly));
-
- // First line is diagonal, second is vertical.
- aPoly.clear();
- aPoly.append(B2DPoint(0, 0));
- aPoly.append(B2DPoint(1, 1));
- aPoly.append(B2DPoint(1, 0));
- CPPUNIT_ASSERT(!utils::containsOnlyHorizontalOrVerticalLines(aPoly));
- }
-
// Change the following lines only, if you add, remove or rename
// member functions of the current class,
// because these macros are need by auto register mechanism.
CPPUNIT_TEST_SUITE(b2dpolygon);
CPPUNIT_TEST(testBasics);
- CPPUNIT_TEST(testContainsOnlyHorizontalOrVerticalLines);
CPPUNIT_TEST_SUITE_END();
}; // class b2dpolygon