Summary: | IMethodBinding.override does not compare method names | ||
---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | Martin Aeschlimann <martinae> |
Component: | Core | Assignee: | Jerome Lanneluc <jerome_lanneluc> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | P3 | CC: | markus.kell.r |
Version: | 3.1 | ||
Target Milestone: | 3.1 RC2 | ||
Hardware: | PC | ||
OS: | Windows XP | ||
Whiteboard: |
Description
Martin Aeschlimann
2005-06-06 06:04:08 EDT
We strictly follow the spec (section 6.4.2 of JLS2) that doesn't take the selector into account. Now is this good or bad ? When renaming a method, a client can currently know in advance if the renamed method will override another method. Do we want to loose this capability ? oh, I see, in fact I didn't read section 6.4.2. The name 'overrides()' is a bit misleading. Would be good if you can add an extra 'Caution!' to the Javadoc comment. Ok to close. Forget comment #1. I was wrong. I re-read the spec and it talks about the method name and signature for override. I'll fix the implementation. +1 for RC2 Changed MethodBinding#overrides(...) to compare the selectors as well. Added regression test CompatibilityRulesTests#test029(). Verified in N20050606-0010 + JDT/Core HEAD *** Bug 80919 has been marked as a duplicate of this bug. *** Verified for 3.1 RC2 using build I20050610-0010 |