RFR 8212137: Remove JrtFileSystem finalize method

2018-12-07 Thread Sundararajan Athijegannathan 

Please review.

Bug: https://bugs.openjdk.java.net/browse/JDK-8212137
Webrev: https://cr.openjdk.java.net/~sundar/8212137/webrev.00/

Thanks,
-Sundar

Re: RFR 8212137: Remove JrtFileSystem finalize method

2018-12-07 Thread 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.

Re: RFR 8212137: Remove JrtFileSystem finalize method

2018-12-07 Thread James Laskey 
+1

Sent from my iPhone

> On Dec 7, 2018, at 4: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/
> 
> Thanks,
> -Sundar

Re: RFR 8212137: Remove JrtFileSystem finalize method

2018-12-07 Thread Mandy Chung 




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