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
