Bug 335751 - [1.7][compiler] Cycle inheritance in type arguments is not detected
Summary: [1.7][compiler] Cycle inheritance in type arguments is not detected
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.7 M6   Edit
Assignee: Srikanth Sankaran CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 334121 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-01-28 21:48 EST by Olivier Thomann CLA
Modified: 2011-03-07 11:35 EST (History)
2 users (show)

See Also:
satyam.kandula: review+

Attachments
Patch under test (7.17 KB, patch)
2011-02-14 05:13 EST, Srikanth Sankaran 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 Olivier Thomann CLA 2011-01-28 21:48:07 EST 
Using HEAD, bug 209149 fixed the forward reference case with compliance 1.7, but cycle should still be reported as an error.

public class X<A extends B, B extends A> {
}

This compiles fine with compliance 1.7.
Comment 1 Olivier Thomann CLA 2011-01-28 21:48:43 EST 
Srikanth, please investigate.
Comment 2 Srikanth Sankaran CLA 2011-02-14 05:13:36 EST 
Created attachment 188887 [details]
Patch under test

See that this patch also fixes bug 334121 and includes a regression
test for it. So I'll be closing the latter as a duplicate.
Comment 3 Srikanth Sankaran CLA 2011-02-14 05:15:19 EST 
*** Bug 334121 has been marked as a duplicate of this bug. ***
Comment 4 Srikanth Sankaran CLA 2011-02-14 09:13:11 EST 
Passes all tests, Satyam please review. TIA.
Comment 5 Satyam Kandula CLA 2011-02-15 00:25:45 EST 
The patch looks good. +1 for it.
Comment 6 Srikanth Sankaran CLA 2011-02-15 00:36:01 EST 
Released in HEAD for 3.7 M6
Comment 7 Ayushman Jain CLA 2011-03-07 11:35:14 EST 
Verified for 3.7M6 using build I20110301-1537.