Bug 165620

Summary:

Regression in 3.3M3 with generics - ambiguous method.

Product:

[Eclipse Project] JDT

Reporter:

R Lenard <rlenard>

Component:

Core

Assignee:

Maxime Daniel <maxime_daniel>

Status:

VERIFIED FIXED

QA Contact:

Severity:

major

Priority:

CC:

kent_johnson

Version:

3.3

Target Milestone:

3.3 M4

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
A code snippet that produces the problem	none
Experimentation - do not release	none
Fix - single variable only	none

Description R Lenard

2006-11-23 01:45:58 EST

Eclipse 3.3M3 complains about code that Eclipse 3.2.1 was happy with and which the JDK is happy with.

The regression makes lots of code with generics ambiguous.

Comment 1 R Lenard

2006-11-23 01:47:37 EST

Created attachment 54401 [details]
A code snippet that produces the problem

Comment 2 R Lenard

2006-11-23 01:48:27 EST

the error is

Severity and Description	Path	Resource	Location	Creation Time	Id
The method ffoo(Iterable<V>) is ambiguous for the type Tester		Tester.java	line 16	1164263973184	2547

Comment 3 Maxime Daniel

2006-11-23 01:56:18 EST

Probably the same root cause as bug 163370.

Comment 4 Philipe Mulet

2006-11-23 03:50:55 EST

Maxime - until resolved, you probably could revert the offending change.

Comment 5 Maxime Daniel

2006-11-23 10:24:50 EST

Released AmbiguousMethodTest#45 into HEAD (inactive) and R3_2_maintenance.

Comment 6 Maxime Daniel

2006-11-23 10:28:35 EST

Created attachment 54419 [details]
Experimentation - do not release

This experimental patch forwards a bit the one submitted for bug 163370 (better management of WildcardBinding-s).
If tests pass, I'll produce a more thorough patch starting from this one.

Comment 7 Maxime Daniel

2006-11-24 01:47:30 EST

Created attachment 54461 [details]
Fix - single variable only

The tests went OK on the experimental patch. This one is better in that method are more carefully distributed amongst classes and more clearly named.
I still need to investigate the multi-variables case.
Kent, would you please have a look and let me know what you think?

Comment 8 Kent Johnson

2006-11-27 11:33:56 EST

Released for 3.3 M4 in HEAD stream

Used the patch for bug 163370

Comment 9 Olivier Thomann

2006-12-11 15:12:07 EST

Verified for 3.3M4 with I20061211-1119