Bug 537917 - ControlUpdater should be API
Summary: ControlUpdater should be API
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.8   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform UI Triaged CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-14 02:37 EDT by Frank Benoit CLA
Modified: 2018-08-14 12:25 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Frank Benoit CLA 2018-08-14 02:37:11 EDT
This class has discouraged access:
org.eclipse.jface.internal.databinding.provisional.swt.ControlUpdater

But it is meant to be used, see e.g.:
https://wiki.eclipse.org/JFace_Data_Binding/TrackedGetter

Please remove the access restriction.
Comment 1 Dani Megert CLA 2018-08-14 11:58:40 EDT
the wiki page says:

"
The data binding framework has internal classes that work similar to ComputedValue and ComputedList, but instead of producing a value or a list, they update widgets. For example, look for ControlUpdater, TableUpdater. These classes are not API yet because we haven't received enough feedback from early adopters yet. If you would like to see this as API, please push for it by filing enhancement requests in bugzilla. 
"

So, do you have any feedback besides requesting it as API?
Comment 2 Frank Benoit CLA 2018-08-14 12:16:23 EDT
Nobody can be an early adaptor, if this page is from December 2007‎. :-D

What feedback is needed?
It works for me.
Comment 3 Dani Megert CLA 2018-08-14 12:25:12 EDT
(In reply to Frank Benoit from comment #2)
> Nobody can be an early adaptor, if this page is from December 2007‎. :-D
> 
> What feedback is needed?
> It works for me.

Yes, it "works" ;-).

The whole provisional package(s) would have to be polished and published, for which we currently don't have resources.

You can continue to use the internal class. You might also suppress the restriction warning.