Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 508579

Summary: Convert the DotExtractor to a top-level class.
Product: [Tools] GEF Reporter: Tamas Miklossy <miklossy>
Component: GEF DOTAssignee: Tamas Miklossy <miklossy>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: unspecified   
Target Milestone: 5.0.0 (Oxygen) M4   
Hardware: PC   
OS: Windows NT   
Whiteboard:

Description Tamas Miklossy CLA 2016-12-02 03:39:50 EST
Currently, the DotExtractor is implemented as a static inner class within the DotGraphView. Since the functionality of the DotExtractor is completely independent of the DotGraphView, the DotExtractor class should be converted into a top-level class.

Moreover, the DotExtractor is a dot concept rather than a dot.ui concept. The newly created DotExtractor class should reside in the org.eclipse.gef.dot plugin (similar to the DotImport and DotExport classes).
Comment 1 Tamas Miklossy CLA 2016-12-02 03:57:08 EST
I pushed the following changes to origin/master:

[508579] Convert the DotExtractor to a top-level class.

- Move the DotExtractor class from the dot.ui plugin into the dot
plugin.
- Adapt the DotExtractor test cases.

Resolving as fixed in 5.0.0 M4.