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

Christian Thalinger christian.thalinger at oracle.com
Fri Sep 28 09:01:31 PDT 2012


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


More information about the hotspot-dev mailing list