Alan, Thank you for the review.
I agree that File.exists is no longer required, but I would prefer to keep it for better readability. -Dmitry On 2015-03-21 13:57, Alan Bateman wrote: > On 21/03/2015 10:50, Dmitry Samersoff wrote: >> Hi Everyone, >> >> Please review small, test only fix. >> >> http://cr.openjdk.java.net/~dsamersoff/JDK-8075569/webrev.01/ >> >> Lock deleted while we are setting last modified time. >> Ignore error and lets the app exits >> > This looks okay to me but since it now handles the lock file going away > at any time then the Files.exists can be removed if you want (it's > harmless to keep it of course). > > -Alan -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.
