Bug 283179 - Adapt Zest to changes in layout
Summary: Adapt Zest to changes in layout
Status: NEW
Alias: None
Product: GEF
Classification: Tools
Component: GEF-Legacy Zest (show other bugs)
Version: 3.6   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: gef-inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 283083
Blocks:
  Show dependency tree
 
Reported: 2009-07-10 15:00 EDT by Mateusz Matela CLA
Modified: 2009-07-13 10:45 EDT (History)
2 users (show)

See Also:


Attachments
patch (119.90 KB, patch)
2009-07-10 15:26 EDT, Mateusz Matela CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mateusz Matela CLA 2009-07-10 15:00:29 EDT
In this bug I'll provide changes necessary to use new layout API provided in bug 283083. I've also let myself clean the code a bit (remove commented and unused code, simplified some unnecessarily complex fragments).
Comment 1 Mateusz Matela CLA 2009-07-10 15:26:51 EDT
Created attachment 141333 [details]
patch
Comment 2 Anthony Hunter CLA 2009-07-10 15:32:54 EDT
A reminder that our next release is a point release 3.6 so we cannot remove any public API.

If you point your API tools to the previous GEF release you can see any API breakage errors.
Comment 3 Ian Bull CLA 2009-07-10 17:20:43 EDT
I thought API was on the bundle level, not project level.  For example, in Eclipse, p2 bundles are marked 1.0, while other bundles (PDE for example) are at 3.5.  So the Zest bundle could be incremented. 

Zest has a small community (compared to GEF for example) and the benefits of cleaning up some of the old API (some of which didn't work anyways), IMHO, outweighs strict API stability.  Now, I'm not suggesting we completely re-write our API, but there are some cases that cleanup will be a welcomed change.

This topic came up on the RT-PMC:
http://dev.eclipse.org/mhonarc/lists/rt-pmc/msg00716.html

and Tom Watson had some good comments here:
http://dev.eclipse.org/mhonarc/lists/rt-pmc/msg00747.html

cheers,
ian
Comment 4 Ian Bull CLA 2009-07-12 16:50:19 EDT
I filed Bug 283252 to discuss the possibility of moving from Zest 1.1 to Zest 2.0.
Comment 5 Anthony Hunter CLA 2009-07-13 10:45:49 EDT
The guidelines here should be to deprecate the old API and add new ones.

Current guidelines from the architecture council is that deprecated API needs to stay in for two years.