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

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 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.