Request for review 8000213: NPG: Should have renamed arrayKlass and typeArrayKlass

[Coleen Phillimore]

Thank you Chris!

coleen

On 9/28/2012 5:42 PM, Christian Thalinger wrote:
> Looks good.  -- Chris
>
> On Sep 28, 2012, at 9:42 AM, Coleen Phillimore<coleen.phillimore at oracle.com>  wrote:
>
>> Christian,  I made that change in the compiler interface files (ci).
>> thanks for pointing out the omission.
>>
>> Hit reload.
>>
>> Coleen
>>
>> On 9/28/2012 12:01 PM, Christian Thalinger wrote:
>>> On Sep 28, 2012, at 6:46 AM, Coleen Phillimore<coleen.phillimore at oracle.com>  wrote:
>>>
>>>> Summary: Capitalize these metadata types (and objArrayKlass)
>>>>
>>>> Also fixed some whitespace errors in classfileParser caused by one of the metadata merges and renames.   Whitespace changes don't really show up in webrev but that's why these files are in the webrev.
>>>>
>>>> open webrev at http://cr.openjdk.java.net/~coleenp/8000213/
>>>> bug link at http://bugs.sun.com/view_bug.do?bug_id=8000213
>>>>
>>>> Sorry in advance for the tedious review.
>>>> Tested against nsk.sajdi.testlist because the SA is sensitive to all changes like this.
>>> Why don't you change the getter methods as well when you're already there:
>>> !   ArrayKlass* get_arrayKlass() {
>>> -- Chris
>>>
>>>> thanks,
>>>> coleen