The changes look ok to me Dan. I know I have used a similar technique elsewhere to handle this situation.
A minor point, and others may have a different opinion but, I don't think the @bug tag needs to list this bug number, since it is not testing a product bug. It is a testcase bug only. But it's not even worth discussing. -Chris > On 1 Nov 2013, at 22:20, Dan Xu <[email protected]> wrote: > > Hi All, > > Please review a simple test fix. In the clean-up stage of MaxPathLength.java > on windows platform, the delete operation may be interfered by anti-virus > software or some Windows services, which will lead to the failure of > recursive directory delete operations. In the change, the test will try its > best to clean up testing folders. If it fails, it will just ignore that, > continue the testing process, and let jtreg to handle the clean-up later. > Thanks! > > Bug: https://bugs.openjdk.java.net/browse/JDK-8027612 > Webrev: http://cr.openjdk.java.net/~dxu/8027612/webrev/ > > -Dan
