RFR [8059563]: (proxy) sun.misc.ProxyGenerator.generateProxyClass should create intermediate directories

Ivan Gerasimov ivan.gerasimov at oracle.com
Tue Oct 7 08:47:36 UTC 2014

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:

Sincerely yours,

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

More information about the core-libs-dev mailing list