[PATCH resend2] 8157758: Use (~0u) instead of (-1) when left-shifting

Max Ockner max.ockner at oracle.com
Thu Jun 23 19:34:20 UTC 2016


Alex,
This change looks good to me.
Thanks,
Max

On 6/22/2016 5:12 PM, Alex Henrie wrote:
> 2016-06-22 14:56 GMT-06:00 Kim Barrett <kim.barrett at oracle.com>:
>>> On Jun 22, 2016, at 3:15 PM, Alex Henrie <alexhenrie24 at gmail.com> wrote:
>>>
>>> 2016-06-15 12:48 GMT-06:00 Kim Barrett <kim.barrett at oracle.com>:
>>>> So I've changed my mind about this patch.  Looks good.
>>> If you approve of the patch, would you please commit it?
>>>
>>> -Alex
>> Hotspot changes generally require at least two reviewers, and at least one must be a “Reviewer”.
>> I only see one so far (me, also a Reviewer).  I’ll poke a couple people about being a second.
>>
>> The change process is described here: http://openjdk.java.net/guide/changePlanning.html
>> The requirement for a second reviewer is a Hotspot extension to that process.
>> Are you covered by an Oracle Contributor Agreement, either individually or through an employer?
>>
>> I can sponsor it once there’s a second review.
> Yes, I have signed the OCA. Is there anything that I can do to help
> find a second reviewer?
>
> -Alex



More information about the hotspot-dev mailing list