Bug 537917

Summary:	ControlUpdater should be API
Product:	[Eclipse Project] Platform	Reporter:	Frank Benoit <frank.rene.benoit>
Component:	UI	Assignee:	Platform UI Triaged <platform-ui-triaged>
Status:	NEW ---	QA Contact:
Severity:	enhancement
Priority:	P3	CC:	daniel_megert
Version:	4.8
Target Milestone:	---
Hardware:	All
OS:	All
Whiteboard:

Description Frank Benoit

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

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

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

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.