RFR(L): 8197405: Improve messages of AbstractMethodErrors and IncompatibleClassChangeErrors.

Lindenmaier, Goetz goetz.lindenmaier at sap.com
Fri Feb 9 14:01:17 UTC 2018


Hi,

I changed the test to use jasm.
http://cr.openjdk.java.net/~goetz/wr18/8197405-ameExMsg/webrev.02/

Also, I fixed a bug and disabled the test on aarch/arm. 

I started to port the simple stuff to aarch/arm. 
I'm happy to include a further patch for these, but if I 
have time I will also try to implement the rest, can't test
it though.

Best regards,
  Goetz.

> -----Original Message-----
> From: David Holmes [mailto:david.holmes at oracle.com]
> Sent: Donnerstag, 8. Februar 2018 23:54
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; hotspot-runtime-
> dev at openjdk.java.net
> Subject: Re: RFR(L): 8197405: Improve messages of AbstractMethodErrors
> and IncompatibleClassChangeErrors.
> 
> Hi Goetz,
> 
> Binary class files can not be checked-in to the repo. You'll need to use
> jasm or jcod files that then get compiled as part of the test.
> 
> Thanks,
> David
> 
> On 9/02/2018 12:41 AM, Lindenmaier, Goetz wrote:
> > Hi,
> >
> > This change improves the messages of AbstractMethodErrors and
> IncompatibleClassChangeErrors.
> > Please review.
> > http://cr.openjdk.java.net/~goetz/wr18/8197405-ameExMsg/webrev.01/
> >
> > It adds tests that check some of the improved messages.
> > The tests check the following improvements:
> >
> > AbstractMethodErrors:
> >
> > test_ameInt, test case 1:
> > before: no message / null
> > after:  Missing implementation of interface method
> MyAbstractInt.anAbstractMethod()Ljava/lang/String;
> >
> > test_ameInt, test case 2:
> > before: no message / null
> > after:  Class ImplementsSomeFunctionsInt does not implement the
> requested method
> >          aFunctionOfMyInterface()Ljava/lang/String; inherited from interface
> MyInterfaceInt1
> >
> > test_ame, interpreted:
> > before: no message / null
> > after:  Missing implementation of interface method
> MyInterface.aFunctionOfMyInterface()V
> >
> > test_ame, compiled:
> > before: MyAbstract.aFunctionOfMyInterface()V
> > after:  Class ImplementsSomeFunctions does not implement the requested
> method
> >          aFunctionOfMyInterface()V inherited from abstract class MyAbstract
> >
> > IncompatibleClassChangeErrors:
> >
> > test_icc_compiled_itable_stub
> > before: vtable stub
> > after:  Class ICC_B does not implement the requested interface ICC_iB
> >
> > Best regards,
> >    Goetz.
> >


More information about the hotspot-runtime-dev mailing list