Bug 5947 - Three entries for 'Parser' in all types list
Summary: Three entries for 'Parser' in all types list
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 2.0   Edit
Hardware: PC Windows 2000
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Claude Knaus CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2001-11-15 07:13 EST by Philipe Mulet CLA
Modified: 2001-11-20 07:08 EST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philipe Mulet CLA 2001-11-15 07:13:25 EST
Build 20011114 - self-hosting scenario

When using the open on type dialog, and enter 'Parser', you will get at least
3 entries for Parser, which are surprisingly not gathered in one set. This 
seems to be caused by the fact the second is an interface, and thus fooling the 
aggregation heuristic.
Comment 1 Erich Gamma CLA 2001-11-15 08:11:53 EST
there are 2 Parser classes on is an interface and the other one is a class.
If we would agreggate we would have to show a combined interface/class icon.
This is confusing and the current behaviour is intentional (we had PRs against 
the other one)

Ok to close?
Comment 2 Philipe Mulet CLA 2001-11-15 17:13:20 EST
No. I understand the distinction in between Class and Interface, but I should 
not have twice a Class entry with the same name, and in my workspace, I have:

"Parser"

[C] Parser
      +- javax.swing.text.html.parser
[I] Parser
      +- org.xml.sax
[C] Parser
      +- org.eclipse.jdt.internal.compiler.parser
      +- org.eclipse.webdav.internal.authentication
      +- org.mozilla.javascript
...

Workspace is available upon request.
Comment 3 Claude Knaus CLA 2001-11-20 07:08:16 EST
fixed in 20011120

The problem was due to the lack of comparability of images. The comparator
could test for equality, but could not assign an order.

The clean fix would be to have the user of the FilterList to define the order 
of the images. For now, an arbitrary order is chosen.