Hi Ivan, This all looks good. +1.
Thanks, Brian On Nov 2, 2016, at 7:39 AM, Ivan Gerasimov <ivan.gerasi...@oracle.com> wrote: > Thank you Brian for review! > > > On 31.10.2016 21:23, Brian Burkhalter wrote: >> One more thing … >> >> In [1] lines 58-59 could be corrected in the same way. > > Yes, right. Thanks for catching it! > > Here's the updated webrev: > http://cr.openjdk.java.net/~igerasim/8168923/01/webrev/ > >> Thanks, >> >> Brian >> >> [1] >> http://hg.openjdk.java.net/jdk9/jdk9/jdk/file/0086eb10182b/src/java.base/share/classes/java/nio/file/TempFileHelper.java >> >> On Oct 31, 2016, at 9:00 AM, Brian Burkhalter <brian.burkhal...@oracle.com> >> wrote: >> >>> Hello, Ivan, >>> >>> Aside from the inevitable copyright year update this looks fine to me. >>> What effect does it have on the regression test run? > > Tested (jdk_io|jdk_nio) on all platforms, no failures.