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

Bug 337640

Summary: [dot4zest] Issues with DOT label parsing
Product: [Tools] GEF Reporter: satyagraha.1956
Component: GEF-Legacy ZestAssignee: gef-inbox <gef-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: clay, nyssen, ram.nath241089, steeg
Version: unspecifiedKeywords: helpwanted
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Illustrates problem. none

Description satyagraha.1956 CLA 2011-02-19 10:38:23 EST
The DOT parser for label= strings does not seem to be properly handling:
1) backslash escaping
2) multiple values using | separator
3) nested values using { }

See the "project" node label value in the attached file, and view the file in ZGRViewer to see what's expected.
Comment 1 satyagraha.1956 CLA 2011-02-19 10:39:06 EST
Created attachment 189349 [details]
Illustrates problem.
Comment 2 Fabian Steeg CLA 2011-02-20 16:07:14 EST
There are actually two different issues here:

The first is escaping in quoted strings. This part is fixed with bug 337639 according to the DOT language spec (http://www.graphviz.org/doc/info/lang.html, in particular the footnote at the bottom): surrounding quotes are removed, and escaped quotes are converted to plain quotes (see TestGraphInstanceDotImport#escapedQuotes).

The second is support for the special record syntax (http://www.graphviz.org/doc/info/shapes.html#record). Supporting that involves both extending the grammar and creating custom figures (contributions welcome, see also bug 321775).
Comment 3 Alexander Nyßen CLA 2014-12-09 13:02:26 EST
Support for record-based shapes is handled in bug #454629 and support for html-like labels is addressed in bug #321775. 

As the initially reported parsing issues have been resolved, closing this as fixed.