Re: RFR: 8248318: Examine the use of boxing in ObjectStreamClass

2021-02-19 Thread Chris Hegarty
On Fri, 19 Feb 2021 10:14:54 GMT, Julia Boes wrote: > This change removes some instances of superfluous boxing in > java.io.ObjectStreamClass. > Testing: tier 1-3 all clear. LGTM. Optionally (and trivially), you could update the JIRA description, from "Examine" to "Remove superfluous use of

Re: RFR: 8248318: Examine the use of boxing in ObjectStreamClass

2021-02-19 Thread Roger Riggs
On Fri, 19 Feb 2021 10:14:54 GMT, Julia Boes wrote: > This change removes some instances of superfluous boxing in > java.io.ObjectStreamClass. > Testing: tier 1-3 all clear. Marked as reviewed by rriggs (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/2641

Re: RFR: 8248318: Examine the use of boxing in ObjectStreamClass

2021-02-19 Thread Daniel Fuchs
On Fri, 19 Feb 2021 10:14:54 GMT, Julia Boes wrote: > This change removes some instances of superfluous boxing in > java.io.ObjectStreamClass. > Testing: tier 1-3 all clear. LGTM. Please add a `noreg-cleanup` label to the JBS issue. - Marked as reviewed by dfuchs (Reviewer). PR:

Re: RFR: 8248318: Examine the use of boxing in ObjectStreamClass

2021-02-19 Thread Jim Laskey
On Fri, 19 Feb 2021 10:14:54 GMT, Julia Boes wrote: > This change removes some instances of superfluous boxing in > java.io.ObjectStreamClass. > Testing: tier 1-3 all clear. Marked as reviewed by jlaskey (Reviewer). - PR: https://git.openjdk.java.net/jdk/pull/2641

Re: RFR: 8248318: Examine the use of boxing in ObjectStreamClass

2021-02-19 Thread Vyom Mani Tewari
On Fri, 19 Feb 2021 10:14:54 GMT, Julia Boes wrote: > This change removes some instances of superfluous boxing in > java.io.ObjectStreamClass. > Testing: tier 1-3 all clear. Looks ok to me. - Marked as reviewed by vyomm...@github.com (no known OpenJDK username). PR:

RFR: 8248318: Examine the use of boxing in ObjectStreamClass

2021-02-19 Thread Julia Boes
This change removes some instances of superfluous boxing in java.io.ObjectStreamClass. Testing: tier 1-3 all clear. - Commit messages: - remove superfluous boxing Changes: https://git.openjdk.java.net/jdk/pull/2641/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=2641=00