[ 
http://issues.apache.org/jira/browse/HARMONY-113?page=comments#action_12367618 
] 

Richard Liang commented on HARMONY-113:
---------------------------------------

Yes, Tim. 
I think you can fix this issue. I will verify your update after you mark this 
issue as fixed. When JIRA57 is acceptted I will create patch for the test 
cases, otherwise there may be duplication. :-)
Thanks a lot.

> java.nio.charset.Charset should regard empty charset name properly as Illegal 
> CharsetName
> -----------------------------------------------------------------------------------------
>
>          Key: HARMONY-113
>          URL: http://issues.apache.org/jira/browse/HARMONY-113
>      Project: Harmony
>         Type: Bug
>   Components: Classlib
>     Reporter: Richard Liang
>  Attachments: charset_patch.txt
>
> I think it's caused by the different between RI 1.4.2 and RI 5.0. Three test 
> cases in JIRA#57  should be updated accordingly.
>       public void testIsSupported_EmptyString() {
>               try {
>                       Charset.isSupported("");
>             fail("Should throw IllegalCharsetNameException!");
>               } catch (IllegalCharsetNameException e) {
>                       //
>               }
>       }
>       public void testConstructor_EmptyCanonicalName() {
>               try {
>                       new MockCharset("", new String[0]);
>             fail("Should throw IllegalCharsetNameException!");
>               } catch (IllegalCharsetNameException e) {
>                       //
>               }
>       }
>       public void testConstructor_EmptyAliases() {
>               try {
>                       new MockCharset("mockChar", new String[] { "" });
>             fail("Should throw IllegalCharsetNameException!");
>               } catch (IllegalCharsetNameException e) {
>                       //
>               }
>       }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to