On 13.12.2012 15:56, Alan Bateman wrote:
On 13/12/2012 11:48, Alexey Utkin wrote:
Here is new version fix:
http://cr.openjdk.java.net/~uta/openjdk-webrevs/JDK-8004928/webrev.02
This looks good to me and thanks for the confirmation that
LoggingDeadlock4 continues to test what it was meant to test.
Minor comment on ClassRestrictions.java is that you've changed the
test to throw Error rather than RuntimeException. Either will cause
the test to fail so it doesn't matter of course.
I did it just for code unification between
test/java/io/Serializable/resolveProxyClass/NonPublicInterface.java
test/java/lang/reflect/Proxy/ClassRestrictions.java
tests. Yes, that need to be mentioned in summary. Sorry.
-uta
