| Summary: | API usage issue in EventDispatcher3D | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Miles Parker <milesparker> | ||||||
| Component: | Gef3d | Assignee: | Jens Von Pilgrim <developer> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||
| Severity: | major | ||||||||
| Priority: | P3 | CC: | developer | ||||||
| Version: | unspecified | ||||||||
| Target Milestone: | --- | ||||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
Miles, thanks for the patch. Actually, I'm already aware of this problem, see https://bugs.eclipse.org/bugs/show_bug.cgi?id=306609 . I haven't submitted the GEF 3.6 related code yet, as Helios is not final yet. I will submit the code as soon as Helios is final. The problem is, that then GEF3D won't compile with GEF 3.5 and elder versions. Also, there seem to be some more problems with GEF 3.6, I have already submitted some fixes, but I haven't had the time to track them all down. Ah -- ok gotcha. That's true even if yo take out the override? Kind of too bad because other than that there is no reason that you can't use GEF3D with 2.5. With GEF 3.6, the EventDispatcher3D could be moved into package o.e.draw3d -- which is nicer of course :-) But yes, it is the very only thing that breaks compatibility with GEF 3.5 (and less). The only solution I can think of is to create two plugin fragments, one for GEF3 3.6 and one of predecessor versions. However, these fragments would contains one class only. Miles, I have committed a version of EventDispatcher3D that should work with GEF 3.5 and GEF 3.6. Looks as if it is quite similar to your patched version. It is the version I'm working with for quite some time, I simply didn't dare to commit it...). Please re-open this bug if the new version doesn't work for you. Created attachment 171215 [details]
mylyn/context/zip
|
Created attachment 171204 [details] Patch to fix method visibility issue It looks like there is an incompatibility in API either in GEF3D or instroduced through GEF 3.6. getFocusOwner needs to be public. (Patch included.) Marking as major just because it won't build on my Helios based setup.