jdk (original) (raw)

Code Review for jdk

Prepared by: prappo on Mon Jul 14 11:51:24 IST 2014
Workspace: /Users/pavelrappo/jdk9/dev/jdk
Compare against: http://hg.openjdk.java.net/jdk9/dev/jdk
Compare against version: 10283
Summary of changes: 198 lines changed: 12 ins; 149 del; 37 mod; 1508 unchg
Changeset: <jdk.changeset>
Legend: Modified file Deleted file New file

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

rev 10284 : 8048175: Remove redundant use of reflection on core classes from JNDI Reviewed-by: duke

12 lines changed: 0 ins; 7 del; 5 mod; 76 unchg

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

rev 10284 : 8048175: Remove redundant use of reflection on core classes from JNDI Reviewed-by: duke

3 lines changed: 0 ins; 1 del; 2 mod; 224 unchg

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

rev 10284 : 8048175: Remove redundant use of reflection on core classes from JNDI Reviewed-by: duke

80 lines changed: 9 ins; 55 del; 16 mod; 956 unchg

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

rev 10284 : 8048175: Remove redundant use of reflection on core classes from JNDI Reviewed-by: duke

24 lines changed: 0 ins; 22 del; 2 mod; 54 unchg

[Cdiffs](src/share/classes/com/sun/jndi/toolkit/corba/CorbaUtils.java.cdiff.html) [Udiffs](src/share/classes/com/sun/jndi/toolkit/corba/CorbaUtils.java.udiff.html) [Sdiffs](src/share/classes/com/sun/jndi/toolkit/corba/CorbaUtils.java.sdiff.html) [Frames](src/share/classes/com/sun/jndi/toolkit/corba/CorbaUtils.java.frames.html) [Old](src/share/classes/com/sun/jndi/toolkit/corba/CorbaUtils.java-.html) [New](src/share/classes/com/sun/jndi/toolkit/corba/CorbaUtils.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/com/sun/jndi/toolkit/corba/CorbaUtils.java) src/share/classes/com/sun/jndi/toolkit/corba/CorbaUtils.java

rev 10284 : 8048175: Remove redundant use of reflection on core classes from JNDI Reviewed-by: duke

79 lines changed: 3 ins; 64 del; 12 mod; 198 unchg


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