BradWalker commented on code in PR #5714:
URL: https://github.com/apache/netbeans/pull/5714#discussion_r1148431161
##########
platform/openide.explorer/src/org/openide/explorer/view/NodeTableModel.java:
##########
@@ -218,9 +218,9 @@ private void computeVisiblePorperties(int visCount) {
int vi = allPropertyColumns[i].getVisibleIndex();
if (vi == -1) {
- sort.put(new Double(i - 0.1), Integer.valueOf(i));
+ sort.put(Double.valueOf(i - 0.1), Integer.valueOf(i));
} else {
- sort.put(new Double(vi), Integer.valueOf(i));
+ sort.put(Double.valueOf(vi), Integer.valueOf(i));
Review Comment:
I looked at the JLS and I'm going to leave it as is if you are okay with
this.
The reason is is the JLS section 5.1.3 Narrowing Primitive Conversions talks
about how valueOf() gets handled. The JLS section 5.5. Casting Contexts handles
type casting and it's not really clear both narrow/expand the cast the same
way. I would really have to look at this in great detail and the work just
didn't justify the brain power. 8-)
So I decided to just leave it alone as this guarantees no change in behavior.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
