Re: RFR(S, TEST) JDK-8075569: jmap test fails due to "ERROR: java.nio.file.NoSuchFileException: 2906081d-06bc-4738-a7e8-f37b8bf13658.lck

2015-03-21 Thread Alan Bateman
On 21/03/2015 13:29, Dmitry Samersoff wrote: 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 Fair enough, either way will work.

Re: RFR(S, TEST) JDK-8075569: jmap test fails due to "ERROR: java.nio.file.NoSuchFileException: 2906081d-06bc-4738-a7e8-f37b8bf13658.lck

2015-03-21 Thread Dmitry Samersoff
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. >> >> h

Re: RFR(S, TEST) JDK-8075569: jmap test fails due to "ERROR: java.nio.file.NoSuchFileException: 2906081d-06bc-4738-a7e8-f37b8bf13658.lck

2015-03-21 Thread Alan Bateman
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

RFR(S, TEST) JDK-8075569: jmap test fails due to "ERROR: java.nio.file.NoSuchFileException: 2906081d-06bc-4738-a7e8-f37b8bf13658.lck

2015-03-21 Thread Dmitry Samersoff
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 -Dmitry -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would