RFR (XS) CR 8014509: @Contended: explicit default value behaves differently from the implicit value

[David Holmes]

Sponsored and submitted to JPRT.

David

On 17/05/2013 7:59 AM, Aleksey Shipilev wrote:
> On 05/17/2013 01:52 AM, Christian Thalinger wrote:
>> Looks good.  -- Chris
>>
>> On May 16, 2013, at 2:25 PM, Aleksey Shipilev <aleksey.shipilev at oracle.com> wrote:
>>
>>> On 05/17/2013 12:50 AM, Vladimir Kozlov wrote:
>>>> You forgot to change != to ==
>>>>
>>>> +        if (_cp->symbol_at(group_index)->utf8_length() != 0) {
>>>> +          group_index = 0; // default contended group
>>>>
>>>> Otherwise it is good. Thank you for the comment.
>>>
>>> Darn. Now I see why test renamings are good: it had actually regressed
>>> the 8003985 test.
>>>
>>> Fixed:
>>>   http://cr.openjdk.java.net/~shade/8014509/webrev.03/
>>>
>>> We need the second reviewer.
>
> Great, thanks.
>
> Any sponsors around? :) Please push:
>   http://cr.openjdk.java.net/~shade/8014509/8014509.changeset
>
> -Aleksey.
>