Issue 78150 - geometry::RealBezierSegment2D needs API change
Summary: geometry::RealBezierSegment2D needs API change
Status: ACCEPTED
Alias: None
Product: App Dev
Classification: Unclassified
Component: api (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-06 16:59 UTC by Armin Le Grand
Modified: 2017-05-20 11:27 UTC (History)
1 user (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 2007-06-06 16:59:57 UTC 
AW: geometry::RealBezierSegment2D uses (P, C1, C2)-style bezier definition. This
is bad for several reasons:
- C1 == P and C2 == P is used to express C1 resp. C2 are unused. This is true
for C1, but for C2 it would need to be expressed with the next P in the next
data section. Because usage of C2==P, a C2 which points (is equal) exactly from
next P to P is not expressable.
- using (Prev, P, Next)-style definition allows that
- it also allows to compute/change continuity with a single segment
- it also allows handling a point and it's controls with a single index

AW->THB: As we discussed, we should change this to a definition which does not
prevent data combinations. For the listed reasons, i propose (Prev, P, Next)-style.
Comment 1 thb 2007-06-06 19:36:24 UTC 
Set proper target
Comment 2 thb 2007-07-27 18:28:52 UTC 
While doing so, also get rid of the tools polygon conversion in vcl::unotools
(hard to maintain, with the changed semantics); and review the places in
cppcanvas, where several back-and-forth conversions between tools polygon and
basegfx polygon take place (e.g. in polypolygonaction.cxx)
Comment 3 thb 2007-12-19 15:55:42 UTC 
Sorry, out of time for 2.4
Comment 4 thb 2008-06-04 16:59:57 UTC 
Retargetting due to resource constraints
Comment 5 thb 2012-07-13 20:48:04 UTC 
Reset to default bug assignee.
Comment 6 Armin Le Grand 2012-07-19 09:56:09 UTC 
ALG: Taking ownership
Comment 7 Marcus 2017-05-20 11:27:49 UTC 
Reset assigne to the default "issues@openoffice.apache.org".