> On 26 Jul 2017, at 12:31, Chris Hegarty <[email protected]> wrote: > > >> On 26 Jul 2017, at 18:03, Hohensee, Paul <[email protected]> wrote: >> >> A double negative (!Files.notExists) somewhat unusual coding and will >> perplex people reading it. They might even switch it back to Files.exists as >> a style cleanup, so I recommend adding a comment explaining why you’re using >> it. > > As the original author of this code, +1000 Code has been pushed. I’ll create new issue for that :)
—Andrei > > -Chris. > > >> Thanks, >> >> Paul >> >> On 7/25/17, 5:08 PM, "core-libs-dev on behalf of Andrey Nazarov" >> <[email protected] on behalf of >> [email protected]> wrote: >> >> Thanks, Brian >> >> —Andrei >>> On 25 Jul 2017, at 16:41, Brian Burkhalter <[email protected]> >>> wrote: >>> >>> Hi Andrei, >>> >>> I think this looks good. >>> >>> Thanks, >>> >>> Brian >>> >>>> On Jul 25, 2017, at 3:59 PM, Andrey Nazarov <[email protected]> >>>> wrote: >>>> >>>> Can anyone look? >>>> >>>> —Thanks, >>>> Andrei >>>>> On 19 Jul 2017, at 18:21, Andrey Nazarov <[email protected]> >>>>> wrote: >>>>> >>>>> Hi, >>>>> >>>>> Could you review fix in test library which should help to resolve >>>>> intermittent issues on Windows machines during directories clean up. >>>>> >>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8184961 >>>>> <https://bugs.openjdk.java.net/browse/JDK-8184961> >>>>> Review: http://cr.openjdk.java.net/~anazarov/JDK-8184961/webrev.00/webrev/ >>>>> >>>>> >>>>> —Andrei >>> >> >> >> >
