Bug 488427 - Update StructuredStateProvider to react to Platform UI API changes
Summary: Update StructuredStateProvider to react to Platform UI API changes
Status: CLOSED FIXED
Alias: None
Product: Dali JPA Tools
Classification: WebTools
Component: JPA (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.6   Edit
Assignee: Brian Vosburgh CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-02-24 20:21 EST by Vaninder Rajput CLA
Modified: 2016-03-02 08:33 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Vaninder Rajput CLA 2016-02-24 20:21:58 EST
Neon build failed after picking up Platform's latest driver. 

Here is the error:

[ERROR] /jobs/genie.webtools/WTP-R3_8_Integration/workspace/webtools.dali/common/plugins/org.eclipse.jpt.common.ui/src/org/eclipse/jpt/common/ui/jface/StructuredStateProvider.java:[25]
[ERROR] public interface StructuredStateProvider
[ERROR] ^^^^^^^^^^^^^^^^^^^^^^^
[ERROR] The default method dispose() inherited from IContentProvider conflicts with another method inherited from IBaseLabelProvider
[ERROR] /jobs/genie.webtools/WTP-R3_8_Integration/workspace/webtools.dali/common/plugins/org.eclipse.jpt.common.ui/src/org/eclipse/jpt/common/ui/jface/TreeStateProvider.java:[25]
[ERROR] public interface TreeStateProvider
[ERROR] ^^^^^^^^^^^^^^^^^
[ERROR] The default method dispose() inherited from IContentProvider conflicts with another method inherited from IBaseLabelProvider


StructuredStateProvider is extending interfaces which are in turn extending IContentProvider and IBaseLabelProvider. dispose() method already existed in IBaseLabelProvider and because of a recent change, this method was added as a default method to IContentProvider. So the code failed to compile because an interface cannot inherit from two interfaces with same default methods. 

https://bugs.eclipse.org/bugs/show_bug.cgi?id=477779

Need to figure out a way to react to the changes.
Comment 1 Brian Vosburgh CLA 2016-02-25 18:19:40 EST
Can you please point me to the appropriate Platform download? Thanks.
Comment 2 Brian Vosburgh CLA 2016-02-26 12:42:48 EST
added default implementation of dispose() to StructuredStateProvider
Comment 3 Vaninder Rajput CLA 2016-03-02 08:33:24 EST
Fixed in the latest build.