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
