remove unnecessary polypolygon subdivide
As the comment already says, it's not necessary. And it is partially responsible for tdf#133016 by splitting an area into smaller parts, which then do not line up completely when AA-ed. Change-Id: If35e2118a0cd9c9b315b9666165c51659a9434e7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94578 Tested-by: Jenkins Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
This commit is contained in:
@@ -322,12 +322,6 @@ namespace vclcanvas
|
||||
::basegfx::B2DPolyPolygon aPolyPoly(
|
||||
::basegfx::unotools::b2DPolyPolygonFromXPolyPolygon2D(xPolyPolygon) );
|
||||
|
||||
if( aPolyPoly.areControlPointsUsed() )
|
||||
{
|
||||
// AW: Not needed for ApplyLineDashing anymore; should be removed
|
||||
aPolyPoly = ::basegfx::utils::adaptiveSubdivideByAngle(aPolyPoly);
|
||||
}
|
||||
|
||||
// apply dashing, if any
|
||||
if( strokeAttributes.DashArray.hasElements() )
|
||||
{
|
||||
|
||||
@@ -223,11 +223,6 @@ namespace slideshow::internal
|
||||
"PathAnimation::PathAnimation(): failed to parse SVG:d path" );
|
||||
ENSURE_OR_THROW( aPolyPoly.count() == 1,
|
||||
"PathAnimation::PathAnimation(): motion path consists of multiple/zero polygon(s)" );
|
||||
|
||||
// TODO(F2): Since getPositionRelative() currently
|
||||
// cannot handle beziers, have to subdivide.
|
||||
// AW: Should be no longer necessary; getPositionRelative is now bezier-safe
|
||||
maPathPoly = ::basegfx::utils::adaptiveSubdivideByAngle(aPolyPoly.getB2DPolygon(0) );
|
||||
}
|
||||
|
||||
virtual ~PathAnimation() override
|
||||
|
||||
@@ -134,11 +134,6 @@ basegfx::B2DPolyPolygon prepareClip( const basegfx::B2DPolyPolygon& rClip )
|
||||
{
|
||||
basegfx::B2DPolyPolygon aClip( rClip );
|
||||
|
||||
// TODO(P2): unnecessary, once XCanvas is correctly handling this
|
||||
// AW: Should be no longer necessary; tools are now bezier-safe
|
||||
if( aClip.areControlPointsUsed() )
|
||||
aClip = basegfx::utils::adaptiveSubdivideByAngle( aClip );
|
||||
|
||||
// normalize polygon, preparation for clipping
|
||||
// in updateCanvas()
|
||||
aClip = basegfx::utils::correctOrientations(aClip);
|
||||
|
||||
@@ -154,11 +154,6 @@ namespace slideshow::internal
|
||||
if (mbFlip)
|
||||
aClipPoly.flip();
|
||||
|
||||
// currently, clipper cannot cope with curves. Subdivide first
|
||||
// AW: Should be no longer necessary; clipping tools are now bezier-safe
|
||||
// if( aClipPoly.areControlPointsUsed() )
|
||||
// aClipPoly = ::basegfx::utils::adaptiveSubdivideByAngle(aClipPoly);
|
||||
|
||||
if( mbSubtractPolygon )
|
||||
{
|
||||
// subtract given polygon from background
|
||||
|
||||
Reference in New Issue
Block a user