RFR 8009581: Xpathexception does not honor initcause()

Aleksej Efimov aleksej.efimov at oracle.com
Tue May 28 12:04:59 UTC 2013


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/

Aleksej

On 05/27/2013 06:57 AM, David Holmes wrote:
> 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