Code Review for open

Prepared by:iignatyev on Wed Sep 26 16:33:19 PDT 2018
Workspace:/Users/iignatye/ws/jdk/jdk/open
Compare against: ssh://hg.openjdk.java.net/jdk/jdk
Compare against version:51880
Summary of changes: 655 lines changed: 239 ins; 355 del; 61 mod; 8301 unchg
Changeset: open.changeset
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/hotspot/jtreg/runtime/Dictionary/ProtectionDomainCacheTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 1 del; 1 mod; 86 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jaxp/javax/xml/jaxp/unittest/catalog/CatalogFileInputTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
6 lines changed: 3 ins; 1 del; 2 mod; 280 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/io/FilePermission/ReadFileOnPath.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
3 lines changed: 2 ins; 0 del; 1 mod; 90 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/io/Serializable/packageAccess/PackageAccessTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 114 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/io/Serializable/resolveClass/consTest/ConsTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 55 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/io/Serializable/resolveClass/consTest/SetupJar.java

rev 51882 : [mq]: 8211171-1
2 lines changed: 2 ins; 0 del; 0 mod; 34 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/io/Serializable/resolveClass/deserializeButton/DeserializeButtonTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 57 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/io/Serializable/superclassDataLoss/SuperclassDataLossTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 119 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/ClassLoader/forNameLeak/ClassForNameLeak.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
5 lines changed: 3 ins; 1 del; 1 mod; 131 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/ClassLoader/getResource/automaticmodules/Driver.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 86 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/ClassLoader/securityManager/ClassLoaderTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
3 lines changed: 1 ins; 1 del; 1 mod; 295 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/Package/IsCompatibleWithDriver.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
3 lines changed: 1 ins; 1 del; 1 mod; 61 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/Package/PackageFromManifest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
2 lines changed: 1 ins; 1 del; 0 mod; 300 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/instrument/executableJAR/ExecJarWithAgent.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 1 del; 1 mod; 137 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/module/AutomaticModulesTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
5 lines changed: 3 ins; 0 del; 2 mod; 1116 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/module/ModuleReader/ModuleReaderTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
5 lines changed: 3 ins; 0 del; 2 mod; 424 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/module/MultiReleaseJarTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
3 lines changed: 1 ins; 0 del; 2 mod; 334 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/lang/module/customfs/ModulesInCustomFileSystem.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
5 lines changed: 3 ins; 0 del; 2 mod; 141 unchg

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

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 1 del; 1 mod; 205 unchg

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

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 1 del; 1 mod; 138 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/nio/channels/AsynchronousChannelGroup/AsExecutor.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
3 lines changed: 1 ins; 0 del; 2 mod; 94 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/nio/channels/AsynchronousChannelGroup/SetupJar.java

rev 51882 : [mq]: 8211171-1
2 lines changed: 2 ins; 0 del; 0 mod; 39 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/nio/charset/spi/CharsetProviderBasicTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
2 lines changed: 0 ins; 1 del; 1 mod; 120 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/nio/charset/spi/SetupJar.java

rev 51882 : [mq]: 8211171-1
2 lines changed: 2 ins; 0 del; 0 mod; 52 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/rmi/module/ModuleTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
5 lines changed: 3 ins; 0 del; 2 mod; 158 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/security/Provider/SecurityProviderModularTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
3 lines changed: 1 ins; 0 del; 2 mod; 383 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/util/ServiceLoader/ModulesTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 486 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/java/util/ServiceLoader/basic/ServiceLoaderBasicTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 160 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/javax/security/auth/login/modules/JaasModularClientTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
3 lines changed: 0 ins; 0 del; 3 mod; 269 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/javax/security/auth/login/modules/JaasModularDefaultHandlerTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
3 lines changed: 1 ins; 0 del; 2 mod; 237 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/jdk/modules/scenarios/automaticmodules/RunWithAutomaticModules.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
5 lines changed: 3 ins; 0 del; 2 mod; 181 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/sun/net/www/protocol/jar/jarbug/TestDriver.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 1 del; 1 mod; 91 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/tools/jlink/basic/BasicTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 161 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/tools/launcher/modules/addexports/manifest/AddExportsAndOpensInManifest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 1 del; 1 mod; 185 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/tools/launcher/modules/addreads/AddReadsTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 227 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/tools/launcher/modules/illegalaccess/IllegalAccessTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 473 unchg

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

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 218 unchg

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

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
4 lines changed: 2 ins; 0 del; 2 mod; 222 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/jdk/tools/launcher/modules/validate/ValidateModulesTest.java

rev 51881 : [mq]: 8211171
rev 51882 : [mq]: 8211171-1
3 lines changed: 1 ins; 0 del; 2 mod; 168 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/lib/jdk/test/lib/util/JarUtils.java

rev 51882 : [mq]: 8211171-1
167 lines changed: 166 ins; 0 del; 1 mod; 174 unchg

------ ------ ------ ------ Old --- ----- --- test/jaxp/javax/xml/jaxp/libs/jaxp/library/JarUtils.java

rev 51881 : [mq]: 8211171
141 lines changed: 0 ins; 141 del; 0 mod; 0 unchg

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

rev 51881 : [mq]: 8211171
202 lines changed: 0 ins; 202 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).