RFR 8025185: MethodHandleInError and MethodTypeInError not handled in ConstantPool::, compare_entry_to and copy_entry_to
John Rose
john.r.rose at oracle.com
Wed Oct 9 21:15:37 PDT 2013
Looks good.
Suggestion: Make normalize_error_tag be static, for namespace cleanliness. Even better, move it into constantTag, as (say) constantTag::non_error_value().
— John
On Oct 9, 2013, at 7:04 PM, Coleen Phillimore <coleen.phillimore at oracle.com> wrote:
>
> Thanks Dan!
> Coleen
>
> On 10/9/2013 6:39 PM, Daniel D. Daugherty wrote:
>> On 10/8/13 12:27 PM, Coleen Phillimore wrote:
>>> Summary: Add missing cases.
>>>
>>> I verified this with the test for https://bugs.openjdk.java.net/browse/JDK-8022701 but don't want to copy the test into with modifications the hotspot repository, still working on refactoring the test for the JDK repository for after this gets pushed.
>>>
>>> open webrev at http://cr.openjdk.java.net/~coleenp/8025185/
>>
>> src/share/vm/oops/constantPool.cpp
>> No comments.
>>
>> Thumbs up.
>>
>> Dan
>>
>>
>>> bug link https://bugs.openjdk.java.net/browse/JDK-8025185
>>>
>>> Thanks,
>>> Coleen
>>>
>>>
>>
>
More information about the serviceability-dev
mailing list