Jason,
You are absolutely right and my fault that I didn't tested the
compatibility of new/old versions.
I have tested it with JDK 8(without fix) as a serializer and JDK 8
(with fix) as deserializer. The behavior was as you expected. Thanks for
catching it.
I'll wait for other comments before generating another webrev.
Aleksej
On 05/28/2013 05:41 PM, Jason Mehrens wrote:
> Alan, David,
> thank you for comments - I also agree with all of them. And as a
result v3:
> http://cr.openjdk.java.net/~dmeetry/8009581/webrev.3/
> >> I think this looks better. I assume that since the
super.getCause() is
> >> null that there is no need to handle IllegalStateException now.
You can still generate a ISE inside readObject by performing "new
XPathException(new Exception()).initCause(null)" in the old code and
then reading that in the new patch. In the new code that line would
fail fast.
Jason
