Review request : 7031166 : pack200 CommandLineTest test failure - simple review

Kumar Srinivasan kumar.x.srinivasan at oracle.COM
Mon Mar 28 01:19:08 UTC 2011


Alan,

http://cr.openjdk.java.net/~ksrini/7031166/webrev.1

Fixed it per your suggestions.

Thanks
Kumar



> Kumar Srinivasan wrote:
>> Hi,
>>
>> Replaces some old nio code which copies over the test sdk to run 
>> tests against.
>> http://cr.openjdk.java.net/~ksrini/7031166/webrev.0/
>>
>> Thanks
>> Kumar
> I see the original code creates the parent directories if they don't 
> exist so if you want it work the same way then you could change it to:
>
> Path parent = dst.toPath().getParent();
> if (parent != null)
>    Files.createDirectories(parent);
> Files.copy(....);
>
> I see that CommandLineTests.runPack200 gets the absolute path before 
> calling this function. You shouldn't need to do that now. I also see 
> it tests if the target file exists and deletes it before the copy. You 
> can skip that too if you add the REPLACE_EXISTING option to the copy. 
> One other suggestion is to do a static import of StandardCopyOption to 
> avoid the copy going into a second line.
>
> -Alan.




More information about the core-libs-dev mailing list