RFR 8009581: Xpathexception does not honor initcause() (original) (raw)
Aleksej Efimov aleksej.efimov at oracle.com
Tue May 28 12:04:59 UTC 2013
- Previous message: RFR 8009581: Xpathexception does not honor initcause()
- Next message: RFR 8009581: Xpathexception does not honor initcause()
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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
- Previous message: RFR 8009581: Xpathexception does not honor initcause()
- Next message: RFR 8009581: Xpathexception does not honor initcause()
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]