Code Review for jdk10

Prepared by:iignatyev on Fri May 26 23:01:48 PDT 2017
Workspace:/Users/iignatye/ws/jdk10/jdk10
Summary of changes: 567 lines changed: 231 ins; 252 del; 84 mod; 8784 unchg
Patch of changes: jdk10.patch
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/com/sun/corba/7130985/CorbaExceptionsCompileTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 0 del; 4 mod; 116 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/io/Serializable/failureAtomicity/FailureAtomicity.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 1 del; 3 mod; 418 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/lang/Class/forName/modules/TestDriver.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 0 del; 4 mod; 168 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/net/URLClassLoader/closetest/CloseTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
3 lines changed: 0 ins; 0 del; 3 mod; 160 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/net/URLClassLoader/closetest/Common.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
2 lines changed: 0 ins; 0 del; 2 mod; 90 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/net/URLClassLoader/closetest/GetResourceAsStream.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
3 lines changed: 0 ins; 1 del; 2 mod; 107 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/net/httpclient/RequestBodyTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
3 lines changed: 0 ins; 0 del; 3 mod; 300 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/net/spi/URLStreamHandlerProvider/Basic.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 0 del; 4 mod; 318 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/nio/file/FileStore/Basic.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
5 lines changed: 0 ins; 2 del; 3 mod; 145 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/nio/file/FileSystem/Basic.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
3 lines changed: 0 ins; 1 del; 2 mod; 125 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/util/logging/FileHandlerMaxLocksTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 1 del; 3 mod; 110 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/util/zip/ZipFile/MultiThreadedReadTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 1 del; 3 mod; 89 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/util/zip/ZipFile/ReadLongZipFileName.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 1 del; 3 mod; 128 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/java/util/zip/ZipFile/ZipEntryFreeTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 1 del; 3 mod; 126 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/jdk/internal/util/jar/TestVersionedStream.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
3 lines changed: 0 ins; 1 del; 2 mod; 224 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/jdk/modules/incubator/ImageModules.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
3 lines changed: 0 ins; 1 del; 2 mod; 381 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jar/InputFilesTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 1 del; 3 mod; 215 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jar/ReleaseBeforeFiles.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 1 del; 3 mod; 161 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jar/compat/CLICompatibility.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 0 del; 4 mod; 653 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jar/mmrjar/Basic.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
3 lines changed: 0 ins; 1 del; 2 mod; 500 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jar/modularJar/Basic.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 1 del; 3 mod; 1169 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jar/multiRelease/ApiValidatorTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
5 lines changed: 1 ins; 2 del; 2 mod; 419 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jar/multiRelease/Basic.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 2 del; 2 mod; 536 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jlink/plugins/SystemModuleDescriptors/CompiledVersionTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 0 del; 4 mod; 129 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jlink/plugins/SystemModuleDescriptors/UserModuleTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 1 del; 3 mod; 320 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jmod/JmodNegativeTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 0 ins; 0 del; 4 mod; 535 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/jmod/JmodTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
5 lines changed: 0 ins; 1 del; 4 mod; 758 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/launcher/LauncherMessageTest.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
4 lines changed: 1 ins; 1 del; 2 mod; 123 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw jdk/test/tools/launcher/modules/patch/systemmodules/PatchSystemModules.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
2 lines changed: 0 ins; 0 del; 2 mod; 261 unchg

------ ------ ------ ------ Old --- ----- --- jdk/test/lib/testlibrary/jdk/testlibrary/FileUtils.java

rev 17249 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
230 lines changed: 0 ins; 230 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New ----- Raw ./test/lib/jdk/test/lib/util/FileUtils.java

rev 2635 : 8180887: move FileUtils to top level testlibrary
Reviewed-by: duke
229 lines changed: 229 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /Users/iignatye/ws/code-tools/webrev/webrev.ksh (vers 25.16-hg+openjdk.java.net).