[PATCH libreoffice-4-0] fdo#53163 fix hatching for objects with curves

David Tardon (via Code Review) gerrit at gerrit.libreoffice.org
Mon Apr 29 04:44:31 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3660

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/60/3660/1

fdo#53163 fix hatching for objects with curves

B2DPolyPolygon can contain Bezier curves, but that is not taken into
account when drawing the hatching in OutputDevice::ImplDrawHatch. The
poly-polygon is adapted already when creating metafile, because the
information about the poly-polygon containing curves is then lost
somewhere in the process. At least, when I do the adaptation in
OutputDevice::ImplDrawHatch, it has no effect and the hatching in the
produced PDF is still wrong.

Change-Id: Ibec9d6aceb23da986fe2d2414dc2e7fd4bf835b4
(cherry picked from commit 092fa33f22bbdcb74c3533750158db1724971878)
---
M drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
1 file changed, 1 insertion(+), 1 deletion(-)



diff --git a/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx b/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
index ada69e6..2127686 100644
--- a/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
+++ b/drawinglayer/source/processor2d/vclmetafileprocessor2d.cxx
@@ -1493,7 +1493,7 @@
 
                     // #i111954# do NOT use decomposition, but use direct VCL-command
                     // process(rCandidate.get2DDecomposition(getViewInformation2D()));
-                    const PolyPolygon aToolsPolyPolygon(aLocalPolyPolygon);
+                    const PolyPolygon aToolsPolyPolygon(basegfx::tools::adaptiveSubdivideByAngle(aLocalPolyPolygon));
                     const HatchStyle aHatchStyle(
                         attribute::HATCHSTYLE_SINGLE == rFillHatchAttribute.getStyle() ? HATCH_SINGLE :
                         attribute::HATCHSTYLE_DOUBLE == rFillHatchAttribute.getStyle() ? HATCH_DOUBLE :

-- 
To view, visit https://gerrit.libreoffice.org/3660
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibec9d6aceb23da986fe2d2414dc2e7fd4bf835b4
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: David Tardon <dtardon at redhat.com>



More information about the LibreOffice mailing list