On Thu, 11 Mar 2021 00:41:48 GMT, Nir Lisker <[email protected]> wrote:
>> Ambarish Rapte has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> correct Float.valueOf()
>
> modules/javafx.web/src/ios/java/javafx/scene/web/JSONDecoder.java line 101:
>
>> 99: long val = Long.parseLong(sNum);
>> 100: if ((val <= Integer.MAX_VALUE) && (Integer.MIN_VALUE <=
>> val)) {
>> 101: return Integer.valueOf(int) val);
>
> Autobox?
I think, when LHS is of type `Object`, it is good to be explicit on creating
Object, it improves readability.
In this case, type of val is `long`, so autobox would create an object of
`Long` not `Integer`.
-------------
PR: https://git.openjdk.java.net/jfx/pull/423