RFR JDK-8209553: ExceptionInInitializerError can have a default detail message if the cause is given

mandy chung mandy.chung at oracle.com
Thu Sep 13 20:50:58 UTC 2018



On 9/13/18 12:47 PM, Jason Mehrens wrote:
> Hi Mandy,
>
> Like in previous patches, I advocated for using 'super.getCause()' in writeObject to avoid any subclass hooking into serialization.  I even lean towards the legacy getXXX methods call super.getCause too so they are compatible with previous behavior.

Good point.  It's probably rare to have subclasses of these types and 
overrides getCause method.  In any case I agree it should call 
super.getCause for correctness.  I also added a test case with 
overridden getCause method.

http://cr.openjdk.java.net/~mchung/jdk12/webrevs/8210721/webrev.01/
> Does Throwable.setCause need a more obscure name encase there are subclasses of throwable in the wild with that signature?

This is package-private method.
> EIIE has double semi-colon in the constructor and PAE has a random javadoc modification.
>

Good catch!  It's cleaned up.

Mandy


More information about the core-libs-dev mailing list