Code Review for jdk

Prepared by:zgu on Wed Sep 2 10:47:52 EDT 2020
Workspace:/home/zgu/ws/jdk8u-dev/jdk
Compare against: ssh://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk
Compare against version:14132
Summary of changes: 887 lines changed: 479 ins; 234 del; 174 mod; 1746 unchg
Changeset: jdk.changeset
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/com/sun/jndi/ldap/Connection.java

rev 14134 : 8062947: Fix exception message to correctly represent LDAP connection failure
Reviewed-by: dfuchs, xyin, vtewari
59 lines changed: 12 ins; 41 del; 6 mod; 992 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/com/sun/jndi/ldap/DefaultLdapDnsProvider.java

rev 14133 : 8151678: com/sun/jndi/ldap/LdapTimeoutTest.java failed due to timeout on DeadServerNoTimeoutTest is incorrect
Reviewed-by: dfuchs, martin, robm
2 lines changed: 0 ins; 0 del; 2 mod; 84 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/com/sun/jndi/ldap/LdapDnsProviderService.java

rev 14133 : 8151678: com/sun/jndi/ldap/LdapTimeoutTest.java failed due to timeout on DeadServerNoTimeoutTest is incorrect
Reviewed-by: dfuchs, martin, robm
16 lines changed: 5 ins; 2 del; 9 mod; 100 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw src/share/classes/com/sun/jndi/ldap/LdapRequest.java

rev 14134 : 8062947: Fix exception message to correctly represent LDAP connection failure
Reviewed-by: dfuchs, xyin, vtewari
23 lines changed: 19 ins; 0 del; 4 mod; 118 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/com/sun/jndi/ldap/LdapTimeoutTest.java

rev 14133 : 8151678: com/sun/jndi/ldap/LdapTimeoutTest.java failed due to timeout on DeadServerNoTimeoutTest is incorrect
Reviewed-by: dfuchs, martin, robm
597 lines changed: 256 ins; 190 del; 151 mod; 103 unchg

Cdiffs Udiffs Sdiffs Frames Old New ----- Raw test/com/sun/jndi/ldap/lib/BaseLdapServer.java

rev 14133 : 8151678: com/sun/jndi/ldap/LdapTimeoutTest.java failed due to timeout on DeadServerNoTimeoutTest is incorrect
Reviewed-by: dfuchs, martin, robm
25 lines changed: 22 ins; 1 del; 2 mod; 349 unchg

------ ------ ------ ------ --- New ----- Raw test/com/sun/jndi/ldap/NamingExceptionMessageTest.java

rev 14134 : 8062947: Fix exception message to correctly represent LDAP connection failure
Reviewed-by: dfuchs, xyin, vtewari
165 lines changed: 165 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /home/zgu/bin/webrev.ksh (vers 25.16-hg+openjdk.java.net).