Bug 195526 - org.eclipse.jdt.core.tests.compiler.regression.GenericTypeTest.test0744 depends on non-specified package-private class HashMap.Entry implementation
Summary: org.eclipse.jdt.core.tests.compiler.regression.GenericTypeTest.test0744 depen...
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3.1   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-07-05 09:44 EDT by Nina Rinskaya CLA
Modified: 2008-09-16 09:46 EDT (History)
2 users (show)

See Also:

Attachments
Proposed fix (2.22 KB, patch)
2007-07-05 10:47 EDT, Olivier Thomann CLA 		no flags Details | Diff
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 Nina Rinskaya CLA 2007-07-05 09:44:48 EDT 
Eclipse Automated test org.eclipse.jdt.core.tests.compiler.regression.GenericTypeTest.test0744 fails on Apache Harmony with "The constructor HashMap.Entry<K2,V2>(int, null, null, null) is undefined" unexpected output because Apache Harmony implementation of java.util.HashMap.Entry (which has package-private access and is not described in J2SE specification) constructor differs from Sun's implementation.
Comment 1 Olivier Thomann CLA 2007-07-05 10:46:26 EDT 
Trivial to fix.
Setting target as 3.3.1.
Comment 2 Olivier Thomann CLA 2007-07-05 10:47:51 EDT 
Created attachment 73124 [details]
Proposed fix

This reports the same error, but doesn't rely on the HashMap.Entry implementation.
Comment 3 Olivier Thomann CLA 2007-07-05 11:09:23 EDT 
Released for 3.4M1.
Comment 4 Olivier Thomann CLA 2007-07-05 11:12:34 EDT 
Released for 3.3.1.
Comment 5 Nina Rinskaya CLA 2007-07-06 02:34:17 EDT 
The issue is not reproducible on Harmony with these changes, thanks.
Comment 6 Olivier Thomann CLA 2007-07-06 09:38:40 EDT 
You might want to use the version from branch R3_3_maintenance.
Comment 7 Nina Rinskaya CLA 2007-07-11 00:01:43 EDT 
I actually don't have the Eclipse Automated sources from CVS, just the sources packed into binary bundle (eclipse-Automated-Tests-3.2.zip/eclipse-Automated-Tests-3.3.zip), so I just copy-pasted the lines that were changed from http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/compiler/regression/GenericTypeTest.java?view=markup, hope that's ok.
Comment 8 Frederic Fusier CLA 2007-08-07 06:41:35 EDT 
Code verified for 3.4M1 using build I20070806-1800.
Comment 9 Eric Jodet CLA 2007-09-03 10:29:09 EDT 
Code verified for 3.3.1 using build M20070831-2000