This is an automated email from the ASF dual-hosted git repository. bwalker pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/netbeans.git
The following commit(s) were added to refs/heads/master by this push: new b75d913bdd Cleanup the use of Double primitive wrapper. b75d913bdd is described below commit b75d913bdd3de6dc4415f3965ac24ba5ed669338 Author: Brad Walker <bwal...@musings.com> AuthorDate: Thu May 25 17:29:35 2023 -0600 Cleanup the use of Double primitive wrapper. Cleanup the use of Double primitive wrapper. As of Java 9, the VM can do a better job of handling the conversion from a Double primitive to a Double Object or vice versa. This is an extension of the work done in #2498. No tests have been touched as those expressly need to manage the conversion. A couple of places I also cleaned up the Float. --- .../validation/constraints/NumberConstraint.java | 2 +- .../validation/constraints/RangeConstraint.java | 10 ++++---- .../project/api/ant/ui/wizard/ImportBuildfile.java | 2 +- .../src/org/netbeans/api/debugger/Properties.java | 2 +- .../bugtracking/issuetable/NodeTableModel.java | 11 ++++---- .../db/dataview/util/DBReadWriteHelper.java | 2 +- .../org/netbeans/lib/ddl/util/PListReader.java | 2 +- .../modules/db/explorer/dlg/CreateTableDialog.java | 2 +- .../editor/completion/PatchedHtmlRenderer.java | 10 ++++---- .../org/netbeans/modules/schema2beans/Common.java | 2 +- .../modules/schema2beans/GraphManager.java | 2 +- .../netbeans/modules/classfile/CPDoubleInfo.java | 2 +- .../modules/form/layoutdesign/LayoutDesigner.java | 30 +++++++++++----------- .../unit/src/org/netbeans/test/stub/api/Stub.java | 4 +-- .../j2seproject/ui/wizards/ImportFoldersPanel.java | 2 +- .../src/swinginterop/SampleTableModel.java | 11 ++++---- .../netbeans/upgrade/systemoptions/SerParser.java | 6 ++--- .../org/netbeans/core/windows/view/ViewHelper.java | 8 +++--- .../core/windows/view/ui/MultiSplitPane.java | 2 +- .../swing/tabcontrol/plaf/BaseTabLayoutModel.java | 4 +-- .../src/org/openide/awt/HtmlRenderer.java | 8 +++--- .../propertysheet/IndexedPropertyEditor.java | 4 +-- .../org/openide/explorer/view/NodeTableModel.java | 13 +++++----- .../src/org/openide/filesystems/XMLMapAttr.java | 2 +- .../profiler/heapwalk/ClassesListController.java | 3 +-- .../websvc/saas/codegen/model/ParameterInfo.java | 2 +- 26 files changed, 75 insertions(+), 73 deletions(-) diff --git a/enterprise/j2ee.sun.ddui/src/org/netbeans/modules/j2ee/sun/validation/constraints/NumberConstraint.java b/enterprise/j2ee.sun.ddui/src/org/netbeans/modules/j2ee/sun/validation/constraints/NumberConstraint.java index 04909c25c4..32721afa83 100644 --- a/enterprise/j2ee.sun.ddui/src/org/netbeans/modules/j2ee/sun/validation/constraints/NumberConstraint.java +++ b/enterprise/j2ee.sun.ddui/src/org/netbeans/modules/j2ee/sun/validation/constraints/NumberConstraint.java @@ -63,7 +63,7 @@ public class NumberConstraint extends ConstraintUtils ArrayList failed_constrained_list = new ArrayList(); if(null != value){ try { - new Double(value); + Double.valueOf(value); return failed_constrained_list; } catch(NumberFormatException e) { String failureMessage = formatFailureMessage(toString(), value, diff --git a/enterprise/j2ee.sun.ddui/src/org/netbeans/modules/j2ee/sun/validation/constraints/RangeConstraint.java b/enterprise/j2ee.sun.ddui/src/org/netbeans/modules/j2ee/sun/validation/constraints/RangeConstraint.java index b5a8cbbfa7..5f818fce79 100644 --- a/enterprise/j2ee.sun.ddui/src/org/netbeans/modules/j2ee/sun/validation/constraints/RangeConstraint.java +++ b/enterprise/j2ee.sun.ddui/src/org/netbeans/modules/j2ee/sun/validation/constraints/RangeConstraint.java @@ -66,8 +66,8 @@ public class RangeConstraint extends ConstraintUtils /** Creates a new instance of <code>RangeConstraint</code>. */ public RangeConstraint(String startValue, String endValue) { try { - this.startValue = new Double(startValue); - this.endValue = new Double(endValue); + this.startValue = Double.valueOf(startValue); + this.endValue = Double.valueOf(endValue); } catch(NumberFormatException e) { String format = BundleReader.getValue("Error_failed_to_create"); //NOI18N @@ -102,7 +102,7 @@ public class RangeConstraint extends ConstraintUtils if((value != null) && (value.length() != 0)){ try { - Double val = new Double(value); + Double val = Double.valueOf(value); if((val.compareTo(startValue) < 0) || (val.compareTo(endValue) > 0)){ addFailure(failed_constrained_list, name, value); @@ -122,7 +122,7 @@ public class RangeConstraint extends ConstraintUtils */ public void setRangeStart(String value){ try { - startValue = new Double(value); + startValue = Double.valueOf(value); } catch(NumberFormatException e) { String format = BundleReader.getValue("Error_failed_to_set"); //NOI18N @@ -142,7 +142,7 @@ public class RangeConstraint extends ConstraintUtils */ public void setRangeEnd(String value){ try { - endValue = new Double(value); + endValue = Double.valueOf(value); } catch(NumberFormatException e) { String format = BundleReader.getValue("Error_failed_to_set"); //NOI18N diff --git a/enterprise/javaee.project/src/org/netbeans/modules/javaee/project/api/ant/ui/wizard/ImportBuildfile.java b/enterprise/javaee.project/src/org/netbeans/modules/javaee/project/api/ant/ui/wizard/ImportBuildfile.java index 7ccdefc84e..84c5145953 100644 --- a/enterprise/javaee.project/src/org/netbeans/modules/javaee/project/api/ant/ui/wizard/ImportBuildfile.java +++ b/enterprise/javaee.project/src/org/netbeans/modules/javaee/project/api/ant/ui/wizard/ImportBuildfile.java @@ -52,7 +52,7 @@ final class ImportBuildfile extends javax.swing.JPanel implements DocumentListen } private void resize() { - int width = (new Double(jLabelDesc.getFontMetrics(jLabelDesc.getFont()).getStringBounds(jLabelDesc.getText(), getGraphics()).getWidth() / 2.7)).intValue() + 40; + int width = (int)(jLabelDesc.getFontMetrics(jLabelDesc.getFont()).getStringBounds(jLabelDesc.getText(), getGraphics()).getWidth() / 2.7) + 40; int height = (jLabelDesc.getFont().getSize() * 5) + 100; if (width < 400) width = 400; diff --git a/ide/api.debugger/src/org/netbeans/api/debugger/Properties.java b/ide/api.debugger/src/org/netbeans/api/debugger/Properties.java index 3f8c7c5822..3f5fc8f7bd 100644 --- a/ide/api.debugger/src/org/netbeans/api/debugger/Properties.java +++ b/ide/api.debugger/src/org/netbeans/api/debugger/Properties.java @@ -1480,7 +1480,7 @@ public abstract class Properties { } else if (classNames[6].equals(className)) { return properties.getFloat(propertyName, 0f); } else if (classNames[7].equals(className)) { - return new Double(properties.getDouble(propertyName, 0d)); + return properties.getDouble(propertyName, 0D); } throw new IllegalArgumentException("Class = '"+className+"'."); } diff --git a/ide/bugtracking.commons/src/org/netbeans/modules/bugtracking/issuetable/NodeTableModel.java b/ide/bugtracking.commons/src/org/netbeans/modules/bugtracking/issuetable/NodeTableModel.java index eaed0b845e..2490c22b6c 100644 --- a/ide/bugtracking.commons/src/org/netbeans/modules/bugtracking/issuetable/NodeTableModel.java +++ b/ide/bugtracking.commons/src/org/netbeans/modules/bugtracking/issuetable/NodeTableModel.java @@ -33,6 +33,7 @@ import java.beans.PropertyChangeListener; import java.util.ArrayList; import java.util.Iterator; +import java.util.Map; import java.util.TreeMap; import javax.swing.JCheckBox; @@ -161,7 +162,7 @@ class NodeTableModel extends AbstractTableModel { int visibleCount = 0; existsComparableColumn = false; - TreeMap<Double, Integer> sort = new TreeMap<Double, Integer>(); + Map<Double, Integer> sort = new TreeMap<>(); int i = 0; int ia = 0; @@ -176,9 +177,9 @@ class NodeTableModel extends AbstractTableModel { Object o = props[i].getValue(ATTR_ORDER_NUMBER); if (o instanceof Integer) { - sort.put(new Double(((Integer) o).doubleValue()), Integer.valueOf(ia)); + sort.put(((Integer)o).doubleValue(), ia); } else { - sort.put(new Double(ia + 0.1), Integer.valueOf(ia)); + sort.put((ia + 0.1), ia); } } else { allPropertyColumns[ia].setVisibleIndex(-1); @@ -231,9 +232,9 @@ class NodeTableModel extends AbstractTableModel { int vi = allPropertyColumns[i].getVisibleIndex(); if (vi == -1) { - sort.put(new Double(i - 0.1), Integer.valueOf(i)); + sort.put((i - 0.1), i); } else { - sort.put(new Double(vi), Integer.valueOf(i)); + sort.put((double)vi, i); } } diff --git a/ide/db.dataview/src/org/netbeans/modules/db/dataview/util/DBReadWriteHelper.java b/ide/db.dataview/src/org/netbeans/modules/db/dataview/util/DBReadWriteHelper.java index 761ee3f61e..f41644c664 100644 --- a/ide/db.dataview/src/org/netbeans/modules/db/dataview/util/DBReadWriteHelper.java +++ b/ide/db.dataview/src/org/netbeans/modules/db/dataview/util/DBReadWriteHelper.java @@ -441,7 +441,7 @@ public class DBReadWriteHelper { } case Types.DOUBLE: - return valueObj instanceof Double ? valueObj : new Double(valueObj.toString()); + return valueObj instanceof Double ? valueObj : Double.valueOf(valueObj.toString()); case Types.FLOAT: case Types.REAL: diff --git a/ide/db/libsrc/org/netbeans/lib/ddl/util/PListReader.java b/ide/db/libsrc/org/netbeans/lib/ddl/util/PListReader.java index 85108d3b29..4abfcc9cd4 100644 --- a/ide/db/libsrc/org/netbeans/lib/ddl/util/PListReader.java +++ b/ide/db/libsrc/org/netbeans/lib/ddl/util/PListReader.java @@ -185,7 +185,7 @@ public class PListReader { tokenizer.pushBack(); double d = Math.rint(tokenizer.nval); if(d == tokenizer.nval) return Integer.valueOf((int)d); - return new Double(tokenizer.nval); + return Double.valueOf(tokenizer.nval); } } diff --git a/ide/db/src/org/netbeans/modules/db/explorer/dlg/CreateTableDialog.java b/ide/db/src/org/netbeans/modules/db/explorer/dlg/CreateTableDialog.java index 93f869ca10..a9a53320ea 100644 --- a/ide/db/src/org/netbeans/modules/db/explorer/dlg/CreateTableDialog.java +++ b/ide/db/src/org/netbeans/modules/db/explorer/dlg/CreateTableDialog.java @@ -471,7 +471,7 @@ public class CreateTableDialog { Map cmap = ColumnItem.getColumnProperty(i); col.setIdentifier(cmap.get("name")); //NOI18N columnName = NbBundle.getMessage (CreateTableDialog.class, "CreateTable_" + i); //NOI18N - columnWidth = (new Double(getFontMetrics(getFont()).getStringBounds(columnName, getGraphics()).getWidth())).intValue() + 20; + columnWidth = (int)(getFontMetrics(getFont()).getStringBounds(columnName, getGraphics()).getWidth()) + 20; if (cmap.containsKey("width")) { // NOI18N if (((Integer)cmap.get("width")).intValue() < columnWidth) col.setPreferredWidth(columnWidth); diff --git a/ide/editor.completion/src/org/netbeans/modules/editor/completion/PatchedHtmlRenderer.java b/ide/editor.completion/src/org/netbeans/modules/editor/completion/PatchedHtmlRenderer.java index dc4680f948..67b3002fab 100644 --- a/ide/editor.completion/src/org/netbeans/modules/editor/completion/PatchedHtmlRenderer.java +++ b/ide/editor.completion/src/org/netbeans/modules/editor/completion/PatchedHtmlRenderer.java @@ -176,7 +176,7 @@ public final class PatchedHtmlRenderer { } double chWidth = wid / chars.length; - int estCharsToPaint = new Double(w / chWidth).intValue(); + int estCharsToPaint = (int)(w / chWidth); if( estCharsToPaint > chars.length ) estCharsToPaint = chars.length; //let's correct the estimate now @@ -841,7 +841,7 @@ public final class PatchedHtmlRenderer { //Word wrap mode goToNextRow = true; - int lastChar = new Double(nextTag - estCharsOver).intValue(); + int lastChar = (int)(nextTag - estCharsOver); //Unlike Swing's word wrap, which does not wrap on tag boundaries correctly, if we're out of space, //we're out of space @@ -892,7 +892,7 @@ public final class PatchedHtmlRenderer { } } else if (brutalWrap) { //wrap without checking word boundaries - length = (new Double((w - widthPainted) / chWidth)).intValue(); + length = (int)((w - widthPainted) / chWidth); if ((pos + length) > nextTag) { length = (nextTag - pos); @@ -911,7 +911,7 @@ public final class PatchedHtmlRenderer { if (strikethrough || underline) { LineMetrics lm = fm.getLineMetrics(chars, pos, length - 1, g); - int lineWidth = new Double(x + r.getWidth()).intValue(); + int lineWidth = (int)(x + r.getWidth()); if (paint) { if (strikethrough) { @@ -930,7 +930,7 @@ public final class PatchedHtmlRenderer { //PENDING - worth supporting with g.setStroke()? A one pixel line is most likely //good enough - //int stThick = new Float (lm.getUnderlineThickness()).intValue(); + //int stThick = Float.valueOf(lm.getUnderlineThickness()).intValue(); g.drawLine(x, y + stPos, lineWidth, y + stPos); } } diff --git a/ide/schema2beans/src/org/netbeans/modules/schema2beans/Common.java b/ide/schema2beans/src/org/netbeans/modules/schema2beans/Common.java index 408a0535d1..fff2741d48 100644 --- a/ide/schema2beans/src/org/netbeans/modules/schema2beans/Common.java +++ b/ide/schema2beans/src/org/netbeans/modules/schema2beans/Common.java @@ -387,7 +387,7 @@ public class Common { case Common.TYPE_FLOAT: return 0.0F; case Common.TYPE_DOUBLE: - return new Double(0.0); + return 0.0D; default: throw new IllegalArgumentException(Common.getMessage( "UnknownType_msg", Integer.valueOf(type))); diff --git a/ide/schema2beans/src/org/netbeans/modules/schema2beans/GraphManager.java b/ide/schema2beans/src/org/netbeans/modules/schema2beans/GraphManager.java index 88f3666ee5..8b28cbf8cd 100644 --- a/ide/schema2beans/src/org/netbeans/modules/schema2beans/GraphManager.java +++ b/ide/schema2beans/src/org/netbeans/modules/schema2beans/GraphManager.java @@ -900,7 +900,7 @@ public class GraphManager extends Object { case Common.TYPE_FLOAT: return 0.0F; case Common.TYPE_DOUBLE: - return new Double(0.0); + return 0.0D; default: throw new IllegalArgumentException(Common.getMessage("UnknownType", type)); } diff --git a/java/classfile/src/org/netbeans/modules/classfile/CPDoubleInfo.java b/java/classfile/src/org/netbeans/modules/classfile/CPDoubleInfo.java index 47540cf7f4..0130a5e113 100644 --- a/java/classfile/src/org/netbeans/modules/classfile/CPDoubleInfo.java +++ b/java/classfile/src/org/netbeans/modules/classfile/CPDoubleInfo.java @@ -32,7 +32,7 @@ public final class CPDoubleInfo extends CPEntry { CPDoubleInfo(ConstantPool pool, double v) { super(pool); - value = new Double(v); + value = v; } /* The VM doesn't allow the next constant pool slot to be used diff --git a/java/form/src/org/netbeans/modules/form/layoutdesign/LayoutDesigner.java b/java/form/src/org/netbeans/modules/form/layoutdesign/LayoutDesigner.java index c869fcc565..6bb705bf57 100644 --- a/java/form/src/org/netbeans/modules/form/layoutdesign/LayoutDesigner.java +++ b/java/form/src/org/netbeans/modules/form/layoutdesign/LayoutDesigner.java @@ -437,8 +437,8 @@ public final class LayoutDesigner implements LayoutConstants { LayoutTestUtils.writeLayoutComponentArray(testCode, "comps", "lc"); //NOI18N LayoutTestUtils.writeRectangleArray(testCode, "bounds", bounds); //NOI18N LayoutTestUtils.writeString(testCode, "defaultContId", defaultContId); //NOI18N - testCode.add("Point hotspot = new Point(" + new Double(hotspot.getX()).intValue() + "," + //NOI18N - new Double(hotspot.getY()).intValue() + ");"); //NOI18N + testCode.add("Point hotspot = new Point(" + (int)(hotspot.getX()) + "," + //NOI18N + (int)(hotspot.getY()) + ");"); //NOI18N testCode.add("ld.startAdding(comps, bounds, hotspot, defaultContId);"); //NOI18N testCode.add("}"); //NOI18N } @@ -463,12 +463,12 @@ public final class LayoutDesigner implements LayoutConstants { if (logTestCode()) { testCode.add("{"); //NOI18N - LayoutTestUtils.writeStringArray(testCode, "compIds", compIds); //NOI18N - LayoutTestUtils.writeRectangleArray(testCode, "bounds", bounds); //NOI18N - testCode.add("Point hotspot = new Point(" + new Double(hotspot.getX()).intValue() + "," + //NOI18N - new Double(hotspot.getY()).intValue() + ");"); //NOI18N - testCode.add("ld.startMoving(compIds, bounds, hotspot);"); //NOI18N - testCode.add("}"); //NOI18N + LayoutTestUtils.writeStringArray(testCode, "compIds", compIds); //NOI18N + LayoutTestUtils.writeRectangleArray(testCode, "bounds", bounds); //NOI18N + testCode.add("Point hotspot = new Point(" + (int)(hotspot.getX()) + "," + //NOI18N + (int)(hotspot.getY()) + ");"); //NOI18N + testCode.add("ld.startMoving(compIds, bounds, hotspot);"); //NOI18N + testCode.add("}"); //NOI18N } setDragTarget(comps[0].getParent(), comps, false); @@ -517,14 +517,14 @@ public final class LayoutDesigner implements LayoutConstants { if (logTestCode()) { testCode.add("{"); //NOI18N - LayoutTestUtils.writeStringArray(testCode, "compIds", compIds); //NOI18N - LayoutTestUtils.writeRectangleArray(testCode, "bounds", bounds); //NOI18N - testCode.add("Point hotspot = new Point(" + new Double(hotspot.getX()).intValue() + "," + //NOI18N - new Double(hotspot.getY()).intValue() + ");"); //NOI18N - LayoutTestUtils.writeIntArray(testCode, "resizeEdges", resizeEdges); //NOI18N - testCode.add("boolean inLayout = " + inLayout + ";"); // NOI18N + LayoutTestUtils.writeStringArray(testCode, "compIds", compIds); //NOI18N + LayoutTestUtils.writeRectangleArray(testCode, "bounds", bounds); //NOI18N + testCode.add("Point hotspot = new Point(" + (int)(hotspot.getX()) + "," + //NOI18N + (int)(hotspot.getY()) + ");"); //NOI18N + LayoutTestUtils.writeIntArray(testCode, "resizeEdges", resizeEdges); //NOI18N + testCode.add("boolean inLayout = " + inLayout + ";"); // NOI18N testCode.add("ld.startResizing(compIds, bounds, hotspot, resizeEdges, inLayout);"); //NOI18N - testCode.add("}"); //NOI18N + testCode.add("}"); //NOI18N } if (inLayout) { diff --git a/java/j2ee.persistence/test/unit/src/org/netbeans/test/stub/api/Stub.java b/java/j2ee.persistence/test/unit/src/org/netbeans/test/stub/api/Stub.java index c0815f5d65..89ec17ca25 100644 --- a/java/j2ee.persistence/test/unit/src/org/netbeans/test/stub/api/Stub.java +++ b/java/j2ee.persistence/test/unit/src/org/netbeans/test/stub/api/Stub.java @@ -157,9 +157,9 @@ public final class Stub { } else if (retClass == Long.TYPE) { return 0L; } else if (retClass == Float.TYPE) { - return Float.valueOf(0); + return 0F; } else if (retClass == Double.TYPE) { - return 0.0; + return 0D; } else if (retClass == Character.TYPE) { return '\0'; } else if (retClass == Boolean.TYPE) { diff --git a/java/java.j2seproject/src/org/netbeans/modules/java/j2seproject/ui/wizards/ImportFoldersPanel.java b/java/java.j2seproject/src/org/netbeans/modules/java/j2seproject/ui/wizards/ImportFoldersPanel.java index cb46343c60..55283a3035 100644 --- a/java/java.j2seproject/src/org/netbeans/modules/java/j2seproject/ui/wizards/ImportFoldersPanel.java +++ b/java/java.j2seproject/src/org/netbeans/modules/java/j2seproject/ui/wizards/ImportFoldersPanel.java @@ -86,7 +86,7 @@ class ImportFoldersPanel extends javax.swing.JPanel { } private void resize() { - int width = (new Double(message.getFontMetrics(message.getFont()).getStringBounds(message.getText(), getGraphics()).getWidth() / 2.7)).intValue() + 40; + int width = (int)(message.getFontMetrics(message.getFont()).getStringBounds(message.getText(), getGraphics()).getWidth() / 2.7) + 40; int height = (message.getFont().getSize() * 5) + 100; if (width < 400) width = 400; diff --git a/javafx/javafx2.samples/SwingInterop/src/swinginterop/SampleTableModel.java b/javafx/javafx2.samples/SwingInterop/src/swinginterop/SampleTableModel.java index 98296f5630..a0357608ae 100644 --- a/javafx/javafx2.samples/SwingInterop/src/swinginterop/SampleTableModel.java +++ b/javafx/javafx2.samples/SwingInterop/src/swinginterop/SampleTableModel.java @@ -35,12 +35,13 @@ public class SampleTableModel extends AbstractTableModel { private static ObservableList<BarChart.Series> bcData; private final String[] names = {"2007", "2008", "2009"}; - + + // TOOD - helpful if these constants had a descriptive constant name private Object[][] data = { - {new Double(567), new Double(956), new Double(1154)}, - {new Double(1292), new Double(1665), new Double(1927)}, - {new Double(1292), new Double(2559), new Double(2774)} - }; + {567D, 956D, 1154D}, + {1292D, 1665D, 1927D}, + {1292D, 2559D, 2774D} + }; public double getTickUnit() { return 1000; diff --git a/nb/o.n.upgrader/src/org/netbeans/upgrade/systemoptions/SerParser.java b/nb/o.n.upgrader/src/org/netbeans/upgrade/systemoptions/SerParser.java index 0d0cecf63d..790468a23f 100644 --- a/nb/o.n.upgrader/src/org/netbeans/upgrade/systemoptions/SerParser.java +++ b/nb/o.n.upgrader/src/org/netbeans/upgrade/systemoptions/SerParser.java @@ -491,7 +491,7 @@ public final class SerParser implements ObjectStreamConstants { makeRef(aw); int size = readInt(); if (size < 0) throw new NotImplementedException(); - aw.values = new ArrayList<Object>(size); + aw.values = new ArrayList<>(size); for (int i = 0; i < size; i++) { if (aw.classdesc.name.equals("[B")) { // NOI18N aw.values.add(readByte()); @@ -504,7 +504,7 @@ public final class SerParser implements ObjectStreamConstants { } else if (aw.classdesc.name.equals("[F")) { // NOI18N aw.values.add(Float.intBitsToFloat(readInt())); } else if (aw.classdesc.name.equals("[D")) { // NOI18N - aw.values.add(new Double(Double.longBitsToDouble(readLong()))); + aw.values.add(Double.longBitsToDouble(readLong())); } else if (aw.classdesc.name.equals("[C")) { // NOI18N aw.values.add(new Character((char)readShort())); } else if (aw.classdesc.name.equals("[Z")) { // NOI18N @@ -568,7 +568,7 @@ public final class SerParser implements ObjectStreamConstants { } else if (fd.type.equals("F")) { // NOI18N values.add(new NameValue(fd, Float.intBitsToFloat(readInt()))); } else if (fd.type.equals("D")) { // NOI18N - values.add(new NameValue(fd, new Double(Double.longBitsToDouble(readLong())))); + values.add(new NameValue(fd, Double.longBitsToDouble(readLong()))); } else if (fd.type.equals("C")) { // NOI18N values.add(new NameValue(fd, new Character((char)readShort()))); } else if (fd.type.equals("Z")) { // NOI18N diff --git a/platform/core.windows/src/org/netbeans/core/windows/view/ViewHelper.java b/platform/core.windows/src/org/netbeans/core/windows/view/ViewHelper.java index e0e3969bbf..be393f39d1 100644 --- a/platform/core.windows/src/org/netbeans/core/windows/view/ViewHelper.java +++ b/platform/core.windows/src/org/netbeans/core/windows/view/ViewHelper.java @@ -165,7 +165,7 @@ final class ViewHelper { List visibleChildren = splitSnapshot.getVisibleChildSnapshots(); ArrayList<ElementAccessor> children = new ArrayList<ElementAccessor>( visibleChildren.size() ); - ArrayList<Double> weights = new ArrayList<Double>( visibleChildren.size() ); + List<Double> weights = new ArrayList<Double>( visibleChildren.size() ); int index = 0; for( Iterator i=visibleChildren.iterator(); i.hasNext(); index++ ) { @@ -183,11 +183,11 @@ final class ViewHelper { double[] weightsToMerge = subSplit.getSplitWeights(); for( int j=0; j<childrenToMerge.length; j++ ) { children.add( childrenToMerge[j] ); - weights.add( Double.valueOf( weightsToMerge[j] * weight ) ); + weights.add(weightsToMerge[j] * weight); } } else { children.add( childAccessor ); - weights.add( Double.valueOf( weight ) ); + weights.add(weight); } } @@ -195,7 +195,7 @@ final class ViewHelper { double[] splitWeights = new double[children.size()]; for( int i=0; i<children.size(); i++ ) { ElementAccessor ea = (ElementAccessor)children.get( i ); - Double weight = (Double)weights.get( i ); + Double weight = weights.get(i); childrenAccessors[i] = ea; splitWeights[i] = weight.doubleValue(); } diff --git a/platform/core.windows/src/org/netbeans/core/windows/view/ui/MultiSplitPane.java b/platform/core.windows/src/org/netbeans/core/windows/view/ui/MultiSplitPane.java index fc4be4c71b..6f9e828be9 100644 --- a/platform/core.windows/src/org/netbeans/core/windows/view/ui/MultiSplitPane.java +++ b/platform/core.windows/src/org/netbeans/core/windows/view/ui/MultiSplitPane.java @@ -189,7 +189,7 @@ public class MultiSplitPane extends JPanel for( int i=0; i<getCellCount(); i++ ) { MultiSplitCell cell = cellAt( i ); double weight = cell.getSize() / size; - splitWeights.add( Double.valueOf( weight ) ); + splitWeights.add(weight); visibleViews.add( cell.getViewElement() ); } } diff --git a/platform/o.n.swing.tabcontrol/src/org/netbeans/swing/tabcontrol/plaf/BaseTabLayoutModel.java b/platform/o.n.swing.tabcontrol/src/org/netbeans/swing/tabcontrol/plaf/BaseTabLayoutModel.java index 44c5a256d4..beba185323 100644 --- a/platform/o.n.swing.tabcontrol/src/org/netbeans/swing/tabcontrol/plaf/BaseTabLayoutModel.java +++ b/platform/o.n.swing.tabcontrol/src/org/netbeans/swing/tabcontrol/plaf/BaseTabLayoutModel.java @@ -123,8 +123,8 @@ class BaseTabLayoutModel implements TabLayoutModel { //No need to calculate for every string String testStr = "Zgj"; //NOI18N Font f = getFont(); - textHeight = new Double(f.getStringBounds(testStr, - BasicScrollingTabDisplayerUI.getOffscreenGraphics(component).getFontRenderContext()).getWidth()).intValue() + 2; + textHeight = (int)(f.getStringBounds(testStr, + BasicScrollingTabDisplayerUI.getOffscreenGraphics(component).getFontRenderContext()).getWidth()) + 2; } return textHeight; } diff --git a/platform/openide.awt/src/org/openide/awt/HtmlRenderer.java b/platform/openide.awt/src/org/openide/awt/HtmlRenderer.java index d7b668c6ab..972b00c462 100644 --- a/platform/openide.awt/src/org/openide/awt/HtmlRenderer.java +++ b/platform/openide.awt/src/org/openide/awt/HtmlRenderer.java @@ -352,7 +352,7 @@ public final class HtmlRenderer { } double chWidth = wid / chars.length; - int estCharsToPaint = new Double(w / chWidth).intValue(); + int estCharsToPaint = (int)(w / chWidth); if( estCharsToPaint > chars.length ) estCharsToPaint = chars.length; //let's correct the estimate now @@ -1057,7 +1057,7 @@ public final class HtmlRenderer { //Word wrap mode goToNextRow = true; - int lastChar = new Double(nextTag - estCharsOver).intValue(); + int lastChar = (int)(nextTag - estCharsOver); //Unlike Swing's word wrap, which does not wrap on tag boundaries correctly, if we're out of space, //we're out of space @@ -1108,7 +1108,7 @@ public final class HtmlRenderer { } } else if (brutalWrap) { //wrap without checking word boundaries - length = (new Double((w - widthPainted) / chWidth)).intValue(); + length = (int)((w - widthPainted) / chWidth); if ((pos + length) > nextTag) { length = (nextTag - pos); @@ -1127,7 +1127,7 @@ public final class HtmlRenderer { if (strikethrough || underline || link) { LineMetrics lm = fm.getLineMetrics(chars, pos, length - 1, g); - int lineWidth = new Double(x + r.getWidth()).intValue(); + int lineWidth = (int)(x + r.getWidth()); if (paint) { if (strikethrough) { diff --git a/platform/openide.explorer/src/org/openide/explorer/propertysheet/IndexedPropertyEditor.java b/platform/openide.explorer/src/org/openide/explorer/propertysheet/IndexedPropertyEditor.java index 160c17355f..6e2ae38043 100644 --- a/platform/openide.explorer/src/org/openide/explorer/propertysheet/IndexedPropertyEditor.java +++ b/platform/openide.explorer/src/org/openide/explorer/propertysheet/IndexedPropertyEditor.java @@ -360,11 +360,11 @@ class IndexedPropertyEditor extends Object implements ExPropertyEditor { } if (getConvertedType().equals(Double.class)) { - value = new Double(0d); + value = 0D; } if (getConvertedType().equals(Float.class)) { - value = 0f; + value = 0F; } if (getConvertedType().equals(Long.class)) { diff --git a/platform/openide.explorer/src/org/openide/explorer/view/NodeTableModel.java b/platform/openide.explorer/src/org/openide/explorer/view/NodeTableModel.java index ddbb06b47b..49d248199a 100644 --- a/platform/openide.explorer/src/org/openide/explorer/view/NodeTableModel.java +++ b/platform/openide.explorer/src/org/openide/explorer/view/NodeTableModel.java @@ -31,6 +31,7 @@ import java.beans.PropertyChangeListener; import java.util.ArrayList; import java.util.Iterator; +import java.util.Map; import java.util.TreeMap; import javax.swing.*; @@ -148,7 +149,7 @@ public class NodeTableModel extends AbstractTableModel { int visibleCount = 0; existsComparableColumn = false; - TreeMap<Double, Integer> sort = new TreeMap<Double, Integer>(); + Map<Double, Integer> sort = new TreeMap<>(); int i = 0; int ia = 0; @@ -163,9 +164,9 @@ public class NodeTableModel extends AbstractTableModel { Object o = props[i].getValue(ATTR_ORDER_NUMBER); if (o instanceof Integer) { - sort.put(new Double(((Integer) o).doubleValue()), Integer.valueOf(ia)); + sort.put(((Integer)o).doubleValue(), ia); } else { - sort.put(new Double(ia + 0.1), new Integer(ia)); + sort.put(ia + 0.1, ia); } } else { allPropertyColumns[ia].setVisibleIndex(-1); @@ -212,15 +213,15 @@ public class NodeTableModel extends AbstractTableModel { private void computeVisiblePorperties(int visCount) { propertyColumns = new int[visCount]; - TreeMap<Double, Integer> sort = new TreeMap<Double, Integer>(); + Map<Double, Integer> sort = new TreeMap<>(); for (int i = 0; i < allPropertyColumns.length; i++) { int vi = allPropertyColumns[i].getVisibleIndex(); if (vi == -1) { - sort.put(new Double(i - 0.1), Integer.valueOf(i)); + sort.put((i - 0.1), i); } else { - sort.put(new Double(vi), Integer.valueOf(i)); + sort.put((double)vi, i); } } diff --git a/platform/openide.filesystems/src/org/openide/filesystems/XMLMapAttr.java b/platform/openide.filesystems/src/org/openide/filesystems/XMLMapAttr.java index 713470671f..86c93f052e 100644 --- a/platform/openide.filesystems/src/org/openide/filesystems/XMLMapAttr.java +++ b/platform/openide.filesystems/src/org/openide/filesystems/XMLMapAttr.java @@ -928,7 +928,7 @@ final class XMLMapAttr implements Map { case 4: return Float.valueOf(value); case 5: - return new Double(value); + return Double.valueOf(value); case 6: return Boolean.valueOf(value); case 7: diff --git a/profiler/profiler.heapwalker/src/org/netbeans/modules/profiler/heapwalk/ClassesListController.java b/profiler/profiler.heapwalker/src/org/netbeans/modules/profiler/heapwalk/ClassesListController.java index beb2573681..77bc414f55 100644 --- a/profiler/profiler.heapwalker/src/org/netbeans/modules/profiler/heapwalk/ClassesListController.java +++ b/profiler/profiler.heapwalker/src/org/netbeans/modules/profiler/heapwalk/ClassesListController.java @@ -164,8 +164,7 @@ public class ClassesListController extends AbstractController { else data[i][4] = (retainedSizeByClass > 0 ? "+" : "") + numberFormat.format(retainedSizeByClass); // NOI18N } } else { - data[i][1] = new Double((double) instancesCount / - (double) totalLiveInstances * 100); + data[i][1] = (double)instancesCount / totalLiveInstances * 100; data[i][2] = numberFormat.format(instancesCount) + " (" // NOI18N + percentFormat.format((double) instancesCount / (double) totalLiveInstances) + ")"; // NOI18N diff --git a/websvccommon/websvc.saas.codegen/src/org/netbeans/modules/websvc/saas/codegen/model/ParameterInfo.java b/websvccommon/websvc.saas.codegen/src/org/netbeans/modules/websvc/saas/codegen/model/ParameterInfo.java index bbc6d122cb..996f80e39f 100644 --- a/websvccommon/websvc.saas.codegen/src/org/netbeans/modules/websvc/saas/codegen/model/ParameterInfo.java +++ b/websvccommon/websvc.saas.codegen/src/org/netbeans/modules/websvc/saas/codegen/model/ParameterInfo.java @@ -185,7 +185,7 @@ public class ParameterInfo { } else if (type == Float.class || type == Float.TYPE) { return 0F; } else if (type == Double.class || type == Double.TYPE) { - return new Double(0); + return 0D; } else if (type == Boolean.class || type == Boolean.TYPE) { return Boolean.FALSE; } else if (type == Character.class || type == Character.TYPE) { --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@netbeans.apache.org For additional commands, e-mail: commits-h...@netbeans.apache.org For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists