[jira] Closed: (HARMONY-5923) [classlib][pack200] CpClass constructor has

["Andrew Cornwall (JIRA)" <ji...@apache.org>]

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

Andrew Cornwall closed HARMONY-5923.
------------------------------------


Patch applied as expected. Thanks!

> [classlib][pack200] CpClass constructor has
> -------------------------------------------
>
>                 Key: HARMONY-5923
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5923
>             Project: Harmony
>          Issue Type: Bug
>          Components: Classlib
>    Affects Versions: 5.0M6
>         Environment: Pack200 head
>            Reporter: Andrew Cornwall
>            Assignee: Sian January
>             Fix For: 5.0M7
>
>         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.