RFR 8212137: Remove JrtFileSystem finalize method

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR 8212137: Remove JrtFileSystem finalize method

sundararajan.athijegannathan
Reply | Threaded
Open this post in threaded view
|

Re: RFR 8212137: Remove JrtFileSystem finalize method

Alan Bateman
On 07/12/2018 08:47, Sundararajan Athijegannathan wrote:
> Please review.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8212137
> Webrev: https://cr.openjdk.java.net/~sundar/8212137/webrev.00/
This looks okay to me.

-Alan.
Reply | Threaded
Open this post in threaded view
|

Re: RFR 8212137: Remove JrtFileSystem finalize method

Jim Laskey (Oracle)
In reply to this post by sundararajan.athijegannathan
+1

Sent from my iPhone

> On Dec 7, 2018, at 4:47 AM, Sundararajan Athijegannathan <[hidden email]> wrote:
>
> Please review.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8212137
> Webrev: https://cr.openjdk.java.net/~sundar/8212137/webrev.00/
>
> Thanks,
> -Sundar

Reply | Threaded
Open this post in threaded view
|

Re: RFR 8212137: Remove JrtFileSystem finalize method

Mandy Chung
In reply to this post by sundararajan.athijegannathan


On 12/7/18 12:47 AM, Sundararajan Athijegannathan wrote:
> Please review.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8212137
> Webrev: https://cr.openjdk.java.net/~sundar/8212137/webrev.00/

Looks okay to me.

Mandy