RFR 8155794 Move Objects.checkIndex BiFunction accepting methods to an internal package
chris.hegarty at oracle.com
Tue May 3 10:14:50 UTC 2016
On 2 May 2016, at 23:37, Paul Sandoz <Paul.Sandoz at oracle.com> wrote:
> Please review:
The changes look fine to me Paul.
> This patch is based on that for 8155258 (VarHandle impl improvements)  sent previously.
> The hotspot changes are really small. Likewise for the 8155258 changes is there is precedent in such cases to push through jdk9-dev rather than hs?
> CCC reviewers strongly indicated for the advanced methods that can customise the exceptions: "You aren't gonna need it”.
> For expediency I propose to move such methods to an internal class jdk.internal.util.Preconditions. I would still like to sweep through java.base and leverage these methods while preserving exception reporting where possible.
> The hotspot changes are just for renaming of the intrinsic method signatures. Since the intrinsic method is now internal i have added an @ForceInline on the corresponding public method, given the potential for this to be used in performance sensitive code.
> JPRT core and hotspot tests pass.
>  http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-May/040740.html <http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-May/040740.html>
More information about the hotspot-dev