> 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
                                          

Reply via email to