RFR(S): 8007986: GrowableArray should implement binary search
roland.westrelin at oracle.com
Tue Feb 23 13:01:31 UTC 2016
I haven’t anything more on this so I’m pushing it. Thanks for looking at this John & Vladimir.
> On Feb 12, 2016, at 4:31 PM, Roland Westrelin <roland.westrelin at oracle.com> wrote:
> Hi John,
> Thanks for looking at this.
>>> overflow issue in the binary search
>> The outer parens look wrong on the fix. You want to cast to int *after* dividing by two, I think.
> You’re right obviously. Is it good to go if I add the missing parentheses?
> int mid = (int)(((uint)max + min) / 2);
More information about the hotspot-compiler-dev