harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Cornwall (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HARMONY-5923) [classlib][pack200] CpClass constructor has
Date Wed, 23 Jul 2008 18:25:31 GMT

     [ https://issues.apache.org/jira/browse/HARMONY-5923?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Andrew Cornwall updated HARMONY-5923:
-------------------------------------

    Attachment: main.patch

main.patch changes the order in the constructor

> [classlib][pack200] CpClass constructor has
> -------------------------------------------
>
>                 Key: HARMONY-5923
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5923
>             Project: Harmony
>          Issue Type: Bug
>         Environment: Pack200 head
>            Reporter: Andrew Cornwall
>         Attachments: main.patch
>
>
> The constructor for CpClass has code that doesn't make sense:
>         this.name = name.underlyingString();
>         this.utf8 = name;
>         if (name == null) {
>             throw new NullPointerException("Null arguments are not allowed");
>         }
> If name.underlyingString() doesn't throw an exception, name can't be null. We should
test before sending the message.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message