Including more reviewer... Thanks, Eric On 2013/3/13 14:28, Eric Wang wrote:
Hi,
Please review the code change, I have updated the test to make sure test only access files and directories created by itself. http://cr.openjdk.java.net/~ewang/8009258/webrev.01/
Here is the execution result: http://cr.openjdk.java.net/~ewang/8009258/GeneralWin32.jtr
Thanks, Eric On 2013/3/5 1:39, Alan Bateman wrote:
On 04/03/2013 17:32, Eric Wang wrote:
Hi,
Please help to review fix below for bug 8009258 <https://jbs.oracle.com/bugs/browse/JDK-8009258>, TEST_BUG:java/io/pathNames/GeneralWin32.java fails intermittently. http://cr.openjdk.java.net/~ewang/8009258/webrev.00/
The File.canRead() method should not be used to check read permission of a directory.
Thanks, Eric I wonder if it would be better to change this test so that it doesn't even attempt to poke around in these directories. I suggest this because there may be other activity going on at the same time. See also 8004096 where the test is running in agentvm mode and is straying into the directory used by another agent VM.
-Alan