On 11.02.16 18:26, Alan Bateman wrote:
On 09/02/2016 17:17, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk9.
The specification is updated, the test will use
IllegalArgumentException instead of IllegalAccessError.
ccc will be filed after the technical review.
Bug: https://bugs.openjdk.java.net/browse/JDK-8130061
Webrev can be found at:
http://cr.openjdk.java.net/~serb/8130061/webrev.00
Should this say newProxyInstance rather than getProxyClass?
The Proxy.newProxyInstance() has not the description of why the
IllegalArgumentException can be thrown, it points to Proxy.getProxyClass():
* <p>{@code Proxy.newProxyInstance} throws
* {@code IllegalArgumentException} for the same reasons that
* {@code Proxy.getProxyClass} does.
......
* @throws IllegalArgumentException if any of the restrictions on the
* parameters that may be passed to {@code getProxyClass}
* are violated
public static Object newProxyInstance(ClassLoader loader,
Class<?>[] interfaces,
InvocationHandler h)
--
Best regards, Sergey.
