Re: RFR(XXS) : 8209386 : [error-prone] StreamResourceLeak in jdk.internal.ed module

2018-08-16 Thread Roger Riggs
Hi Igor, Looks fine. Thanks for the fix,  Roger On 8/10/18 4:44 PM, Igor Ignatyev wrote: http://cr.openjdk.java.net/~iignatyev//8209386/webrev.00/index.html 4 lines changed: 1 ins; 0 del; 3 mod; Hi all, could you please review this small fix for the error detected in jdk.internal.ed by

Re: RFR(XXS) : 8209386 : [error-prone] StreamResourceLeak in jdk.internal.ed module

2018-08-15 Thread Igor Ignatyev
ping? -- Igor > On Aug 10, 2018, at 1:44 PM, Igor Ignatyev wrote: > > http://cr.openjdk.java.net/~iignatyev//8209386/webrev.00/index.html >> 4 lines changed: 1 ins; 0 del; 3 mod; > > Hi all, > > could you please review this small fix for the error detected in > jdk.internal.ed by

RFR(XXS) : 8209386 : [error-prone] StreamResourceLeak in jdk.internal.ed module

2018-08-10 Thread Igor Ignatyev
http://cr.openjdk.java.net/~iignatyev//8209386/webrev.00/index.html > 4 lines changed: 1 ins; 0 del; 3 mod; Hi all, could you please review this small fix for the error detected in jdk.internal.ed by error-prone? webrev: http://cr.openjdk.java.net/~iignatyev//8209386/webrev.00/index.html JBS: