JDK-8103780 : Remove SortableList and FilterableList
  • Type: Task
  • Component: javafx
  • Sub-Component: base
  • Affected Version: fx2.0
  • Priority: P2
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2011-07-25
  • Updated: 2015-06-16
  • Resolved: 2011-08-05
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
fx2.0Fixed
Related Reports
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Blocks :  
Relates :  
Description
We want to remove the interfaces FilterableList and SortableList, as well as the implementations FilteredList and SortedList from the public API. These are optional features, that are not part of the committed deliverables for the Presidio release, but they were added very late during the beta timeframe.

The reasons are:
1. The current functionality is insufficient. We know that for sure already. It is not clear, if the missing functionality can be implemented compatible to the current design.
2. The performance is not were we want it to be for a release.
3. Stability seems to be an issue.
Comments
OK / SQE
04-08-2011

OK / Arch
28-07-2011

OK / RM
28-07-2011

OK / Dev (For Nandini Ramani)
28-07-2011

In addition to removing the classes mentioned above, we want to do add the following methods in ListChangeListener.Change to enable implementations of SortedList and FilteredList with better performance. public boolean next(); // moves to the next change public void reset(); // moves the internal pointer back to the first change If an implementation of ListChangeListener currently looks like this: new ListChangeListener () { @Override public void onChanged(Change c) { // some code } }; It has to be updated like this: new ListChangeListener () { @Override public void onChanged(Change c) { while (c.next()) { // some code } } }; All implementations of ListChangeListener.Change in our repository will be updated as part of this API change.
28-07-2011

Discussed this on architecture alias. No objection to this change.
28-07-2011

CCR / SQE: ok, about 20 affected tests. 1md to update tests would be required.
28-07-2011