RFR: 8252180: [JEP 390] Deprecate wrapper class constructors for removal

Dan Smith dlsmith at openjdk.java.net
Sat Oct 10 06:32:21 UTC 2020


On Fri, 9 Oct 2020 23:54:01 GMT, Mandy Chung <mchung at openjdk.org> wrote:

>> When the code is changed to not use the constructors, the warning (deprecation and remove) should be removed.
>> Some of the code is upstream and requires more changes.
>
> We should change `MemberName` to call the factory method `Byte::valueof` (that's JDK code).
> 
> For the Graal change, you are touching the code anyway -- I think changing them to use the factory methods would be
> better than adding "removal" to the suppression.   But it may be better to check if these tests  intentionally test
> with the public constructors to test for JIT optimization before applying the change.

Yeah, Mandy replied to the question I should have asked: for all the suppressed "removal" points, what is the plan to
fix the offending code? If not now, when? ("Later" may be a reasonable answer, but fleshing out the followup tasks
would be helpful.)

-------------

PR: https://git.openjdk.java.net/valhalla/pull/221



More information about the valhalla-dev mailing list