Please review the updated webrev:
http://cr.openjdk.java.net/~amlu/8181394/webrev.01

CreateFileTree is also used by other tests, no change.

Thanks,
Amy

On 6/1/17 6:51 PM, Alan Bateman wrote:
webrev: http://cr.openjdk.java.net/~amlu/8181394/webrev.00/
I think it would be cleaner if you merged FindTest, CreateFileTree and PrintFileTree. That should eliminate the need to launch PrintFileTree in its own VM, it would make the test setup simpler too. Also it would be nice to avoid SerializationUtils too.

-Alan


Reply via email to