Issue 112442 - unify svg:d im/exporters
Summary: unify svg:d im/exporters
Status: CONFIRMED
Alias: None
Product: Draw
Classification: Application
Component: code (show other issues)
Version: DEV300m81
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-16 12:36 UTC by Armin Le Grand
Modified: 2017-05-20 11:33 UTC (History)
5 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description Armin Le Grand 2010-06-16 12:36:50 UTC 
AW: Currently we have (a) the SdXMLImExSvgDElement helper in xmloff which
converts between com::sun::star::drawing::PointSequenceSequence,
com::sun::star::drawing::FlagSequenceSequence and svg:d string and (b)
importFromSvgD/exportToSvgD in basegfx which convert directly between
B2DPolyPolygon and svg:d string.

There are reasons to change that:
- functionality is double
- (b) supports svg:d's 'a'/'A' statement
- (b) supports double precision

There are reasons to do that for 3.4 (not 3.3):
- too dangerous to change now, should be done after next split
- double precision (not yet in API, though)
- use B2DPolyPolygon to scale plygons to svg:ViewBox (there is a task asking for
this, too)
- need to support svg:d's 'a'/'A' (#i41438#)
Comment 1 Armin Le Grand 2010-06-16 12:51:02 UTC 
AW: Hint: there are alredy converters
SvxConvertPolyPolygonBezierToB2DPolyPolygon and
SvxConvertB2DPolyPolygonToPolyPolygonBezier in svx, should move to basegfx
(canvastools.hxx/cxx?).
Comment 2 thb 2010-06-18 21:24:23 UTC 
@aw: let's move the discussion from issue 41438 over here - no, did not do very
thorough testing of my patch, besides my usual bunch of Draw files. Especially
did not check roundtripping/round-off/older versions. Patch needs to be hoisted
to dev300 (can do over WE), but kinda missed feature freeze for 3.3 is next
week, so maybe 3.4 is indeed a better idea.
Comment 3 thb 2010-06-18 21:25:45 UTC 
repeating patch url here: http://users.freedesktop.org/~thorsten/svg-compat.diff
Comment 4 Armin Le Grand 2010-06-21 11:41:40 UTC 
AW: Okay, target 3.4 agreed...
Comment 5 Armin Le Grand 2010-06-21 11:43:26 UTC 
AW->CL: Any plans to get API for polygons to double precision...?
Comment 6 Armin Le Grand 2012-01-16 14:26:04 UTC 
ALG: There is one more svg:d creator in filter/source/svg/svgwriter.cxx for Svg export.
Comment 7 Regina Henschel 2012-04-01 11:14:05 UTC 
Changing Product to Drawing to make it more visible in searches.
Comment 8 Marcus 2017-05-20 11:33:40 UTC 
Reset assigne to the default "issues@openoffice.apache.org".