On 09/05/2013 06:27, Henry Jen wrote:
On 05/08/2013 09:17 PM, Mike Duigou wrote:
Looks good to me.
Thanks, we have a minor update to exclude changes for DirectoryStream.
http://cr.openjdk.java.net/~henryjen/ccc/8006884.2/webrev/
A quick review on the update is really appreciated.
Looks good to me.
Just on the FaultFileSystem used by the tests. A typo in the class
description " A FileSystem help testing". For completeness then .close
should be idempotent. Minor alignment problem in getFileStore.
Thanks for fixing the bug in the PassThroughFileSystem, that method must
not be used by the other tests that uses this test file system.
-Alan
