RFR 8009581: Xpathexception does not honor initcause() (original) (raw)

David Holmes david.holmes at oracle.com
Mon May 27 02:57:49 UTC 2013

On 26/05/2013 1:41 AM, Alan Bateman wrote:

On 25/05/2013 09:37, Aleksej Efimov wrote:

David, Jason, Thank you for your comments and suggestions. They all were taken in account and as a result - the new webrev: http://cr.openjdk.java.net/~dmeetry/8009581/webrev.2/ I think this looks better. I assume that since the super.getCause() is null that there is no need to handle IllegalStateException now.

I would agree on both counts.

David

I think the test could be beefed up to cover the serialization/de-serialization too (since this is new code and doesn't appear to be exercised by other tests).

-Alan

More information about the core-libs-dev mailing list