On 10/6/2014 4:35 AM, Ivan Gerasimov wrote:
Hello!
This is a 7u-only fix.
Some users complain about the ProxyGenerator not creating the
intermediate directories when asked to keep the generated files.
It throws an IO exception instead
The fix is essentially a part of JDK-8004260.
That was a relatively big update, which required CCC, so it's not
feasible to port it as a whole.
Would you please help review it?
BUGURL: https://bugs.openjdk.java.net/browse/JDK-8059563
WEBREV: http://cr.openjdk.java.net/~igerasim/8059563/0/webrev/
The fix looks okay.
Typo in the new test line 26: s/intermidiate/intermediate. It would be
good to make sure the directory doesn't exist before generating the
proxy class and also check if the expected file is created after it's
generated. An alternative is to call Proxy.getProxyClass instead
ProxyGenerator as your test is under java/lang/reflect/Proxy. Maybe the
testname be renamed to "SaveProxyClassFileTest.java"
Mandy
