Bug 483839 - Add @Deprecated to TreePathViewerSorter and offer replacement
Summary: Add @Deprecated to TreePathViewerSorter and offer replacement
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.3   Edit
Hardware: PC All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 395213 402464
Blocks: 482137
  Show dependency tree
 
Reported: 2015-12-07 14:21 EST by Lars Vogel CLA
Modified: 2015-12-07 14:21 EST (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Lars Vogel CLA 2015-12-07 14:21:27 EST 
+++ This bug was initially created as a clone of Bug #402464 +++

From https://bugs.eclipse.org/bugs/show_bug.cgi?id=402464#c12

>  ** @deprecated see <code>ViewerSorter</code> for more information.
>...
> public class TreePathViewerSorter extends ViewerSorter {

That doesn't make sense and violates our rules about API deprecation.

ViewerSorter doesn't tell what clients of TreePathViewerSorter should do. If you want to deprecate TreePathViewerSorter, you have to implement a new class TreePathViewerComparator and port at least one client as a proof that it really works.