RFR: 8147588: Jar file and Zip file not removed in spite of the OPEN_DELETE flag
Alan Bateman
Alan.Bateman at oracle.com
Tue May 17 12:14:35 UTC 2016
On 15/05/2016 19:54, Xueming Shen wrote:
> Thanks Alan,
>
> Here is the webrev with the first proposed change.
>
> http://cr.openjdk.java.net/~sherman/8147588/webrev/
>
> -sherman
>
It needs a bit of clean-up but I think we can live with the approach.
One thing that could be cleaned up is to move the loading of RAF to
SharedSecrets.getJavaIORandomAccessFileAccess using ensuredInitialized.
Another suggestion is to rename to newRandomAccess with the 3 parameters
as a flags parameter to allow additional flags be passed in. This would
allow the ZipFile.Source change to be simply:
String flags = (toDelete) ? "t" : "";
this.zfile = SharedSecrets.getJavaIORandomAccessFileAccess()
.newRandomAccessFile(key.file, mode, flags);
It would need isWindows and so would be a lot cleaner (I think).
-Alan
More information about the core-libs-dev
mailing list