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