Deprecation warnings
Doug Simon
doug.simon at oracle.com
Wed Feb 1 19:13:04 UTC 2017
> On 1 Feb 2017, at 19:55, Tom Rodriguez <tom.rodriguez at oracle.com> wrote:
>
>
>
> Doug Simon wrote:
>>> On 1 Feb 2017, at 19:42, Andrew Haley<aph at redhat.com> wrote:
>>>
>>> /nfs/zebedee/home/graal/aph/truffle/truffle/com.oracle.truffle.sl/src/com/oracle/truffle/sl/parser/Scanner.java:388: warning: [deprecation] Integer(int) in Integer has been deprecated
>>> literals.put("return", new Integer(17));
>>>
>>> There are too many of these. Shall I just fix them all?
>>
>> No - we’re stuck with those because these methods are not deprecated on jdk8. So if you use SuppressWarnings, you’ll get a warning on jdk8 about an unnecessary suppression!
>
> Well they can be changed to use valueOf instead, though for test cases I think we'll need to make sure we don't actually care about object identity. Those I think we're stuck with. That's going to be a very painful deprecation warning for lots of people.
Yes, we can change the ones where identity doesn’t matter. I hadn’t looked closely and assumed most uses require identity semantics.
-Doug
More information about the graal-dev
mailing list