Thank you Mandy!
I've updated the webrev based on your suggestions.
The test is moved to test/sun/misc/ProxyGenerator/ directory.
Here's the updated webrev:
http://cr.openjdk.java.net/~igerasim/8059563/1/webrev/
Sincerely yours,
Ivan
On 07.10.2014 9:47, Mandy Chung wrote:
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
