8000763: use XXX.valueOf methods instead of constructors
Reviewed-by: mchung, forax
--- a/jdk/src/share/classes/com/sun/rowset/CachedRowSetImpl.java Thu Oct 11 18:24:38 2012 +0100
+++ b/jdk/src/share/classes/com/sun/rowset/CachedRowSetImpl.java Thu Oct 11 18:46:31 2012 -0400
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2003, 2011, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2012, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -1746,12 +1746,7 @@
// convert to a Double and compare to zero
try {
- Double d = new Double(value.toString());
- if (d.compareTo(new Double((double)0)) == 0) {
- return false;
- } else {
- return true;
- }
+ return Double.compare(Double.parseDouble(value.toString()), 0) != 0;
} catch (NumberFormatException ex) {
throw new SQLException(MessageFormat.format(resBundle.handleGetObject("cachedrowsetimpl.boolfail").toString(),
new Object[] {value.toString().trim(), columnIndex}));
@@ -4432,7 +4427,7 @@
// make sure the cursor is on a valid row
checkCursor();
- Object obj = convertNumeric(new Float(x),
+ Object obj = convertNumeric(Float.valueOf(x),
java.sql.Types.REAL,
RowSetMD.getColumnType(columnIndex));
@@ -4467,7 +4462,7 @@
checkIndex(columnIndex);
// make sure the cursor is on a valid row
checkCursor();
- Object obj = convertNumeric(new Double(x),
+ Object obj = convertNumeric(Double.valueOf(x),
java.sql.Types.DOUBLE,
RowSetMD.getColumnType(columnIndex));
--- a/jdk/src/share/classes/com/sun/rowset/FilteredRowSetImpl.java Thu Oct 11 18:24:38 2012 +0100
+++ b/jdk/src/share/classes/com/sun/rowset/FilteredRowSetImpl.java Thu Oct 11 18:46:31 2012 -0400
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2003, 2010, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2012, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -839,7 +839,7 @@
if(onInsertRow) {
if(p != null) {
- bool = p.evaluate(new Float(x) , columnIndex);
+ bool = p.evaluate(Float.valueOf(x), columnIndex);
if(!bool) {
throw new SQLException(resBundle.handleGetObject("filteredrowsetimpl.notallowed").toString());
@@ -906,7 +906,7 @@
if(onInsertRow) {
if(p != null) {
- bool = p.evaluate(new Double(x) , columnIndex);
+ bool = p.evaluate(Double.valueOf(x) , columnIndex);
if(!bool) {
throw new SQLException(resBundle.handleGetObject("filteredrowsetimpl.notallowed").toString());
--- a/jdk/src/share/classes/javax/sql/rowset/BaseRowSet.java Thu Oct 11 18:24:38 2012 +0100
+++ b/jdk/src/share/classes/javax/sql/rowset/BaseRowSet.java Thu Oct 11 18:46:31 2012 -0400
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2003, 2011, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2003, 2012, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -1850,7 +1850,7 @@
if(params == null){
throw new SQLException("Set initParams() before setFloat");
}
- params.put(Integer.valueOf(parameterIndex - 1), new Float(x));
+ params.put(Integer.valueOf(parameterIndex - 1), Float.valueOf(x));
}
/**
@@ -1882,7 +1882,7 @@
if(params == null){
throw new SQLException("Set initParams() before setDouble");
}
- params.put(Integer.valueOf(parameterIndex - 1), new Double(x));
+ params.put(Integer.valueOf(parameterIndex - 1), Double.valueOf(x));
}
/**
--- a/jdk/src/share/classes/javax/sql/rowset/serial/SQLOutputImpl.java Thu Oct 11 18:24:38 2012 +0100
+++ b/jdk/src/share/classes/javax/sql/rowset/serial/SQLOutputImpl.java Thu Oct 11 18:46:31 2012 -0400
@@ -215,7 +215,7 @@
*/
@SuppressWarnings("unchecked")
public void writeFloat(float x) throws SQLException {
- attribs.add(new Float(x));
+ attribs.add(Float.valueOf(x));
}
/**
@@ -230,7 +230,7 @@
*/
@SuppressWarnings("unchecked")
public void writeDouble(double x) throws SQLException{
- attribs.add(new Double(x));
+ attribs.add(Double.valueOf(x));
}
/**