Bug 165620 - Regression in 3.3M3 with generics - ambiguous method.
Summary: Regression in 3.3M3 with generics - ambiguous method.
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 major (vote)
Target Milestone: 3.3 M4   Edit
Assignee: Maxime Daniel CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-11-23 01:45 EST by R Lenard CLA
Modified: 2006-12-11 15:12 EST (History)
1 user (show)

See Also:

Attachments
A code snippet that produces the problem (688 bytes, text/plain)
2006-11-23 01:47 EST, R Lenard CLA 		no flags Details
Experimentation - do not release (6.13 KB, patch)
2006-11-23 10:28 EST, Maxime Daniel CLA 		no flags Details | Diff
Fix - single variable only (7.56 KB, patch)
2006-11-24 01:47 EST, Maxime Daniel CLA 		no flags Details | Diff
Show Obsolete (1) View All 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 R Lenard CLA 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 CLA 2006-11-23 01:47:37 EST 
Created attachment 54401 [details]
A code snippet that produces the problem
Comment 2 R Lenard CLA 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 CLA 2006-11-23 01:56:18 EST 
Probably the same root cause as bug 163370.
Comment 4 Philipe Mulet CLA 2006-11-23 03:50:55 EST 
Maxime - until resolved, you probably could revert the offending change.
Comment 5 Maxime Daniel CLA 2006-11-23 10:24:50 EST 
Released AmbiguousMethodTest#45 into HEAD (inactive) and R3_2_maintenance.
Comment 6 Maxime Daniel CLA 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 CLA 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 CLA 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 CLA 2006-12-11 15:12:07 EST 
Verified for 3.3M4 with I20061211-1119