Bug 224468 - Request: Draw2D enhancements
Summary: Request: Draw2D enhancements
Status: RESOLVED FIXED
Alias: None
Product: GEF
Classification: Tools
Component: GEF-Legacy Draw2d (show other bugs)
Version: 3.4   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: 3.4.0 (Ganymede) M6   Edit
Assignee: Alex Boyko CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-03-27 16:54 EDT by Alex Boyko CLA
Modified: 2008-09-18 13:38 EDT (History)
2 users (show)

See Also:


Attachments
patch (9.48 KB, patch)
2008-03-27 17:05 EDT, Alex Boyko CLA
ahunter.eclipse: iplog+
aboyko: review?
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Boyko CLA 2008-03-27 16:54:27 EDT
1. Integer overflow avoiding fix to distance between two points
2. Integer overflow avoiding fix for Geomtery#linesIntersect(...)
3. Moved left and right node references from virtual node to node and added punblic getters to access left and right nodes from the client layer. All nodes have references to left and right side nodes intialized in PopulateRanks graph visitor and not only virtual nodes. This change is important for graph layout post processing procedures such as alternative edge routing.
4. TransposeMetrics graph visitor should transpose virtual node dimensions.
5. PrecisionRectangle#setSize(Dimension d) is added to handle PrecisionDimension

Patch will be attached
Comment 1 Alex Boyko CLA 2008-03-27 17:05:44 EDT
Created attachment 93877 [details]
patch

Fixes for the described issues
Comment 2 Anthony Hunter CLA 2008-03-31 11:55:15 EDT
Hi Alex,

ok with the minor changes to the org.eclipse.draw2d.geometry and  org.eclipse.draw2d.graph. Committed to HEAD.

We cannot export Export-Package: org.eclipse.draw2d.examples.graph

org.eclipse.draw2d.examples.graph is not actually in the GEF builds, so it makes no sense to export it. I think I also have a concern with making the examples "API" by exporting them.

As an aside, I know you are likely looking at reusing the draw2d examples in GMF. We already get grief for having the GMF examples/tests depend on the EMF examples, so bad idea starting down the road of depending on the GEF examples. We are actually going to move the GMF examples to not use the EMF examples. So ok to do what you need to not depend on the GEF examples :-)
Comment 3 Anthony Hunter CLA 2008-03-31 11:56:09 EDT
ugh, reopen to assign the bug to Alex and not the inbox.
Comment 4 Anthony Hunter CLA 2008-03-31 11:56:53 EDT
(In reply to comment #2)
> ok with the minor changes to the org.eclipse.draw2d.geometry and 
> org.eclipse.draw2d.graph. Committed to HEAD.