open (original) (raw)

Code Review for open

Prepared by: robm on Wed Dec 6 17:43:09 GMT 2017
Workspace: /tank/ws/next/open
Compare against: http://hg.openjdk.java.net/jdk/jdk/
Summary of changes: 535 lines changed: 491 ins; 6 del; 38 mod; 636 unchg
Patch of changes: <open.patch>
Legend: Modified file Deleted file New file

[Cdiffs](src/java.base/share/classes/module-info.java.cdiff.html) [Udiffs](src/java.base/share/classes/module-info.java.udiff.html) [Sdiffs](src/java.base/share/classes/module-info.java.sdiff.html) [Frames](src/java.base/share/classes/module-info.java.frames.html) [Old](src/java.base/share/classes/module-info.java-.html) [New](src/java.base/share/classes/module-info.java.html) [Patch](src/java.base/share/classes/module-info.java.patch) [Raw](raw%5Ffiles/new/src/java.base/share/classes/module-info.java) src/java.base/share/classes/module-info.java

1 line changed: 1 ins; 0 del; 0 mod; 363 unchg

[Cdiffs](src/java.naming/share/classes/com/sun/jndi/ldap/LdapCtxFactory.java.cdiff.html) [Udiffs](src/java.naming/share/classes/com/sun/jndi/ldap/LdapCtxFactory.java.udiff.html) [Sdiffs](src/java.naming/share/classes/com/sun/jndi/ldap/LdapCtxFactory.java.sdiff.html) [Frames](src/java.naming/share/classes/com/sun/jndi/ldap/LdapCtxFactory.java.frames.html) [Old](src/java.naming/share/classes/com/sun/jndi/ldap/LdapCtxFactory.java-.html) [New](src/java.naming/share/classes/com/sun/jndi/ldap/LdapCtxFactory.java.html) [Patch](src/java.naming/share/classes/com/sun/jndi/ldap/LdapCtxFactory.java.patch) [Raw](raw%5Ffiles/new/src/java.naming/share/classes/com/sun/jndi/ldap/LdapCtxFactory.java) src/java.naming/share/classes/com/sun/jndi/ldap/LdapCtxFactory.java

80 lines changed: 36 ins; 6 del; 38 mod; 220 unchg

[Cdiffs](src/java.naming/share/classes/module-info.java.cdiff.html) [Udiffs](src/java.naming/share/classes/module-info.java.udiff.html) [Sdiffs](src/java.naming/share/classes/module-info.java.sdiff.html) [Frames](src/java.naming/share/classes/module-info.java.frames.html) [Old](src/java.naming/share/classes/module-info.java-.html) [New](src/java.naming/share/classes/module-info.java.html) [Patch](src/java.naming/share/classes/module-info.java.patch) [Raw](raw%5Ffiles/new/src/java.naming/share/classes/module-info.java) src/java.naming/share/classes/module-info.java

2 lines changed: 2 ins; 0 del; 0 mod; 53 unchg

------ ------ ------ ------ ---[New](src/java.naming/share/classes/com/sun/jndi/ldap/DefaultLdapDnsProvider.java.html) [Patch](src/java.naming/share/classes/com/sun/jndi/ldap/DefaultLdapDnsProvider.java.patch) [Raw](raw%5Ffiles/new/src/java.naming/share/classes/com/sun/jndi/ldap/DefaultLdapDnsProvider.java) src/java.naming/share/classes/com/sun/jndi/ldap/DefaultLdapDnsProvider.java

52 lines changed: 52 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ---[New](src/java.naming/share/classes/com/sun/jndi/ldap/LdapDnsProviderService.java.html) [Patch](src/java.naming/share/classes/com/sun/jndi/ldap/LdapDnsProviderService.java.patch) [Raw](raw%5Ffiles/new/src/java.naming/share/classes/com/sun/jndi/ldap/LdapDnsProviderService.java) src/java.naming/share/classes/com/sun/jndi/ldap/LdapDnsProviderService.java

87 lines changed: 87 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ---[New](src/java.naming/share/classes/javax/naming/spi/ldap/LdapDnsProvider.java.html) [Patch](src/java.naming/share/classes/javax/naming/spi/ldap/LdapDnsProvider.java.patch) [Raw](raw%5Ffiles/new/src/java.naming/share/classes/javax/naming/spi/ldap/LdapDnsProvider.java) src/java.naming/share/classes/javax/naming/spi/ldap/LdapDnsProvider.java

61 lines changed: 61 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ---[New](src/java.naming/share/classes/javax/naming/spi/ldap/LdapDnsProviderResult.java.html) [Patch](src/java.naming/share/classes/javax/naming/spi/ldap/LdapDnsProviderResult.java.patch) [Raw](raw%5Ffiles/new/src/java.naming/share/classes/javax/naming/spi/ldap/LdapDnsProviderResult.java) src/java.naming/share/classes/javax/naming/spi/ldap/LdapDnsProviderResult.java

45 lines changed: 45 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ---[New](test/jdk/com/sun/jndi/ldap/LdapDnsProviderTest.java.html) [Patch](test/jdk/com/sun/jndi/ldap/LdapDnsProviderTest.java.patch) [Raw](raw%5Ffiles/new/test/jdk/com/sun/jndi/ldap/LdapDnsProviderTest.java) test/jdk/com/sun/jndi/ldap/LdapDnsProviderTest.java

189 lines changed: 189 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ---[New](test/jdk/com/sun/jndi/ldap/dnsprovider/TestDnsProvider.java.html) [Patch](test/jdk/com/sun/jndi/ldap/dnsprovider/TestDnsProvider.java.patch) [Raw](raw%5Ffiles/new/test/jdk/com/sun/jndi/ldap/dnsprovider/TestDnsProvider.java) test/jdk/com/sun/jndi/ldap/dnsprovider/TestDnsProvider.java

17 lines changed: 17 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ ---[New](test/jdk/com/sun/jndi/ldap/javax.naming.spi.ldap.LdapDnsProvider.html) [Patch](test/jdk/com/sun/jndi/ldap/javax.naming.spi.ldap.LdapDnsProvider.patch) [Raw](raw%5Ffiles/new/test/jdk/com/sun/jndi/ldap/javax.naming.spi.ldap.LdapDnsProvider) test/jdk/com/sun/jndi/ldap/javax.naming.spi.ldap.LdapDnsProvider

1 line changed: 1 ins; 0 del; 0 mod; 0 unchg


This code review page was prepared using /home/robm/sync/scripts/webrev/webrev.ksh(vers 25.15-hg+openjdk.java.net).