RFR: 8032010: Attempt to resolve abstract method in concrete class fails with AbstractMethodError

Calvin Cheung calvin.cheung at oracle.com
Wed Feb 5 13:25:20 PST 2014


Thanks for your review, Coleen.

Calvin

On 2/5/2014 1:18 PM, Coleen Phillimore wrote:
>
> This looks good.
> Coleen
>
> On 1/31/14 5:45 PM, Calvin Cheung wrote:
>> This change, by removing a check in LinkResolver::resolve_method(), 
>> is to conform with the recent change in JVMS-8 [1].
>> The following check during method resolution is removed:
>> JVMS-5.4.3.3. Method Resolution
>> "If method lookup succeeds and the method is abstract, but C is not 
>> abstract, method resolution throws an AbstractMethodError."
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8032010
>>
>> webrev: http://cr.openjdk.java.net/~ccheung/8032010/webrev/
>>
>> tests: (on linux-x64)
>>     vm.quick
>>     jck vm and lang
>>     jtreg on java/util/stream and the new testcase
>>     defmeth
>>     jprt
>>
>> thanks,
>> Calvin
>>
>> [1] http://cr.openjdk.java.net/~dlsmith/jsr335-0.9.1.html#J
>



More information about the hotspot-dev mailing list