jdk (original) (raw)
Code Review for jdk
Prepared by: | prappo on Tue Jul 1 12:33:43 IST 2014 |
---|---|
Workspace: | /Users/pavelrappo/jdk9/dev/jdk |
Compare against: | http://hg.openjdk.java.net/jdk9/dev/jdk |
Compare against version: | 10201 |
Summary of changes: | 49 lines changed: 0 ins; 2 del; 47 mod; 26473 unchg |
Changeset: | <jdk.changeset> |
Legend: | Modified file Deleted file New file |
[Cdiffs](src/share/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java.cdiff.html) [Udiffs](src/share/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java.udiff.html) [Sdiffs](src/share/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java.sdiff.html) [Frames](src/share/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java.frames.html) [Old](src/share/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java-.html) [New](src/share/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java)
src/share/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
2 lines changed: 0 ins; 1 del; 1 mod; 2564 unchg
[Cdiffs](src/share/classes/com/sun/jmx/snmp/SnmpString.java.cdiff.html) [Udiffs](src/share/classes/com/sun/jmx/snmp/SnmpString.java.udiff.html) [Sdiffs](src/share/classes/com/sun/jmx/snmp/SnmpString.java.sdiff.html) [Frames](src/share/classes/com/sun/jmx/snmp/SnmpString.java.frames.html) [Old](src/share/classes/com/sun/jmx/snmp/SnmpString.java-.html) [New](src/share/classes/com/sun/jmx/snmp/SnmpString.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/com/sun/jmx/snmp/SnmpString.java)
src/share/classes/com/sun/jmx/snmp/SnmpString.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
1 line changed: 0 ins; 0 del; 1 mod; 277 unchg
[Cdiffs](src/share/classes/com/sun/jndi/toolkit/dir/SearchFilter.java.cdiff.html) [Udiffs](src/share/classes/com/sun/jndi/toolkit/dir/SearchFilter.java.udiff.html) [Sdiffs](src/share/classes/com/sun/jndi/toolkit/dir/SearchFilter.java.sdiff.html) [Frames](src/share/classes/com/sun/jndi/toolkit/dir/SearchFilter.java.frames.html) [Old](src/share/classes/com/sun/jndi/toolkit/dir/SearchFilter.java-.html) [New](src/share/classes/com/sun/jndi/toolkit/dir/SearchFilter.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/com/sun/jndi/toolkit/dir/SearchFilter.java)
src/share/classes/com/sun/jndi/toolkit/dir/SearchFilter.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
1 line changed: 0 ins; 0 del; 1 mod; 676 unchg
[Cdiffs](src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java.cdiff.html) [Udiffs](src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java.udiff.html) [Sdiffs](src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java.sdiff.html) [Frames](src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java.frames.html) [Old](src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java-.html) [New](src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java)
src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
1 line changed: 0 ins; 0 del; 1 mod; 1625 unchg
[Cdiffs](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Client.java.cdiff.html) [Udiffs](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Client.java.udiff.html) [Sdiffs](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Client.java.sdiff.html) [Frames](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Client.java.frames.html) [Old](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Client.java-.html) [New](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Client.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Client.java)
src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Client.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
2 lines changed: 0 ins; 0 del; 2 mod; 327 unchg
[Cdiffs](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Server.java.cdiff.html) [Udiffs](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Server.java.udiff.html) [Sdiffs](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Server.java.sdiff.html) [Frames](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Server.java.frames.html) [Old](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Server.java-.html) [New](src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Server.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Server.java)
src/share/classes/com/sun/security/sasl/gsskerb/GssKrb5Server.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
2 lines changed: 0 ins; 0 del; 2 mod; 363 unchg
[Cdiffs](src/share/classes/com/sun/security/sasl/util/AbstractSaslImpl.java.cdiff.html) [Udiffs](src/share/classes/com/sun/security/sasl/util/AbstractSaslImpl.java.udiff.html) [Sdiffs](src/share/classes/com/sun/security/sasl/util/AbstractSaslImpl.java.sdiff.html) [Frames](src/share/classes/com/sun/security/sasl/util/AbstractSaslImpl.java.frames.html) [Old](src/share/classes/com/sun/security/sasl/util/AbstractSaslImpl.java-.html) [New](src/share/classes/com/sun/security/sasl/util/AbstractSaslImpl.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/com/sun/security/sasl/util/AbstractSaslImpl.java)
src/share/classes/com/sun/security/sasl/util/AbstractSaslImpl.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
1 line changed: 0 ins; 0 del; 1 mod; 364 unchg
[Cdiffs](src/share/classes/java/awt/Component.java.cdiff.html) [Udiffs](src/share/classes/java/awt/Component.java.udiff.html) [Sdiffs](src/share/classes/java/awt/Component.java.sdiff.html) [Frames](src/share/classes/java/awt/Component.java.frames.html) [Old](src/share/classes/java/awt/Component.java-.html) [New](src/share/classes/java/awt/Component.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/java/awt/Component.java)
src/share/classes/java/awt/Component.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
1 line changed: 0 ins; 0 del; 1 mod; 10304 unchg
[Cdiffs](src/share/classes/java/awt/image/renderable/ParameterBlock.java.cdiff.html) [Udiffs](src/share/classes/java/awt/image/renderable/ParameterBlock.java.udiff.html) [Sdiffs](src/share/classes/java/awt/image/renderable/ParameterBlock.java.sdiff.html) [Frames](src/share/classes/java/awt/image/renderable/ParameterBlock.java.frames.html) [Old](src/share/classes/java/awt/image/renderable/ParameterBlock.java-.html) [New](src/share/classes/java/awt/image/renderable/ParameterBlock.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/java/awt/image/renderable/ParameterBlock.java)
src/share/classes/java/awt/image/renderable/ParameterBlock.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
6 lines changed: 0 ins; 0 del; 6 mod; 722 unchg
[Cdiffs](src/share/classes/java/beans/Statement.java.cdiff.html) [Udiffs](src/share/classes/java/beans/Statement.java.udiff.html) [Sdiffs](src/share/classes/java/beans/Statement.java.sdiff.html) [Frames](src/share/classes/java/beans/Statement.java.frames.html) [Old](src/share/classes/java/beans/Statement.java-.html) [New](src/share/classes/java/beans/Statement.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/java/beans/Statement.java)
src/share/classes/java/beans/Statement.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
1 line changed: 0 ins; 0 del; 1 mod; 362 unchg
[Cdiffs](src/share/classes/javax/management/loading/MLet.java.cdiff.html) [Udiffs](src/share/classes/javax/management/loading/MLet.java.udiff.html) [Sdiffs](src/share/classes/javax/management/loading/MLet.java.sdiff.html) [Frames](src/share/classes/javax/management/loading/MLet.java.frames.html) [Old](src/share/classes/javax/management/loading/MLet.java-.html) [New](src/share/classes/javax/management/loading/MLet.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/javax/management/loading/MLet.java)
src/share/classes/javax/management/loading/MLet.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
2 lines changed: 0 ins; 0 del; 2 mod; 1336 unchg
[Cdiffs](src/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java.cdiff.html) [Udiffs](src/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java.udiff.html) [Sdiffs](src/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java.sdiff.html) [Frames](src/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java.frames.html) [Old](src/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java-.html) [New](src/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java)
src/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
5 lines changed: 0 ins; 1 del; 4 mod; 2520 unchg
[Cdiffs](src/share/classes/sun/management/snmp/jvminstr/JvmThreadInstanceEntryImpl.java.cdiff.html) [Udiffs](src/share/classes/sun/management/snmp/jvminstr/JvmThreadInstanceEntryImpl.java.udiff.html) [Sdiffs](src/share/classes/sun/management/snmp/jvminstr/JvmThreadInstanceEntryImpl.java.sdiff.html) [Frames](src/share/classes/sun/management/snmp/jvminstr/JvmThreadInstanceEntryImpl.java.frames.html) [Old](src/share/classes/sun/management/snmp/jvminstr/JvmThreadInstanceEntryImpl.java-.html) [New](src/share/classes/sun/management/snmp/jvminstr/JvmThreadInstanceEntryImpl.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/sun/management/snmp/jvminstr/JvmThreadInstanceEntryImpl.java)
src/share/classes/sun/management/snmp/jvminstr/JvmThreadInstanceEntryImpl.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
1 line changed: 0 ins; 0 del; 1 mod; 329 unchg
[Cdiffs](src/share/classes/sun/misc/ProxyGenerator.java.cdiff.html) [Udiffs](src/share/classes/sun/misc/ProxyGenerator.java.udiff.html) [Sdiffs](src/share/classes/sun/misc/ProxyGenerator.java.sdiff.html) [Frames](src/share/classes/sun/misc/ProxyGenerator.java.frames.html) [Old](src/share/classes/sun/misc/ProxyGenerator.java-.html) [New](src/share/classes/sun/misc/ProxyGenerator.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/sun/misc/ProxyGenerator.java)
src/share/classes/sun/misc/ProxyGenerator.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
2 lines changed: 0 ins; 0 del; 2 mod; 2029 unchg
[Cdiffs](src/share/classes/sun/security/pkcs/PKCS9Attribute.java.cdiff.html) [Udiffs](src/share/classes/sun/security/pkcs/PKCS9Attribute.java.udiff.html) [Sdiffs](src/share/classes/sun/security/pkcs/PKCS9Attribute.java.sdiff.html) [Frames](src/share/classes/sun/security/pkcs/PKCS9Attribute.java.frames.html) [Old](src/share/classes/sun/security/pkcs/PKCS9Attribute.java-.html) [New](src/share/classes/sun/security/pkcs/PKCS9Attribute.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/sun/security/pkcs/PKCS9Attribute.java)
src/share/classes/sun/security/pkcs/PKCS9Attribute.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
18 lines changed: 0 ins; 0 del; 18 mod; 860 unchg
[Cdiffs](src/share/classes/sun/security/x509/AVA.java.cdiff.html) [Udiffs](src/share/classes/sun/security/x509/AVA.java.udiff.html) [Sdiffs](src/share/classes/sun/security/x509/AVA.java.sdiff.html) [Frames](src/share/classes/sun/security/x509/AVA.java.frames.html) [Old](src/share/classes/sun/security/x509/AVA.java-.html) [New](src/share/classes/sun/security/x509/AVA.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/sun/security/x509/AVA.java)
src/share/classes/sun/security/x509/AVA.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
1 line changed: 0 ins; 0 del; 1 mod; 1355 unchg
[Cdiffs](src/share/classes/sun/tools/jconsole/inspector/Utils.java.cdiff.html) [Udiffs](src/share/classes/sun/tools/jconsole/inspector/Utils.java.udiff.html) [Sdiffs](src/share/classes/sun/tools/jconsole/inspector/Utils.java.sdiff.html) [Frames](src/share/classes/sun/tools/jconsole/inspector/Utils.java.frames.html) [Old](src/share/classes/sun/tools/jconsole/inspector/Utils.java-.html) [New](src/share/classes/sun/tools/jconsole/inspector/Utils.java.html) -----[Raw](raw%5Ffiles/new/src/share/classes/sun/tools/jconsole/inspector/Utils.java)
src/share/classes/sun/tools/jconsole/inspector/Utils.java
rev 10202 : 8048874: Replace uses of 'new Byte', 'new Short' and 'new Character' with appropriate alternative across core classes Reviewed-by: chegar, prappo Contributed-by: Otavio Santana otaviojava@java.net
2 lines changed: 0 ins; 0 del; 2 mod; 460 unchg
This code review page was prepared using /Users/pavelrappo/code-tools/webrev/webrev.ksh(vers 25.2-hg+openjdk.java.net).