----- Mail original ----- > De: "Paul Sandoz" <[email protected]> > Cc: "hotspot-dev developers" <[email protected]>, "Core-Libs-Dev" > <[email protected]> > Envoyé: Mardi 3 Mai 2016 22:03:12 > Objet: Re: RFR 8155794 Move Objects.checkIndex BiFunction accepting methods > to an internal package > > > > On 3 May 2016, at 07:05, Remi Forax <[email protected]> wrote: > > > > Looks good to me. > > minor nitpick, neither checkFromToIndex() nor checkFromIndexSize() are > > instrinsic (now !) but i think they also should be annotated with > > @ForceInline in j.u.Objects to avoid the code to be asymmetric or weird if > > one of these methods is intrinsinfied later. > > > > I was tempted, but for the moment i am inclined to leave as is and revisit > if/when necessary. Trying to be conservative when using those annotations.
Ok, in fact i wonder if @ForceInline on checkIndex is even useful because the VM considers the implementation as trivial, so it will be routinely inlined. > > Thanks, > Paul. > cheers, Rémi
