Hello Tim,
I'm wondering why I did not just copy the first sentence. :-)
"A charset name **must** begin with either a letter or a digit." Does this
mean if the charset name which begin with neither a letter nor a digit should be regarded
as an illegal charset name?
Richard Liang
China Software Development Lab, IBM
Tim Ellison wrote:
Richard Liang wrote:
Hello Tim,
I think this is caused by different understanding of the java spec:
A charset name **must** begin with either a letter or a digit. The empty
string is not a legal charset name....
What do think the implication of "must" here? :-)
But the name isn't empty, it is "-UTF-8" ? I must be missing something...
Regards,
Tim
Tim Ellison (JIRA) wrote:
[
http://issues.apache.org/jira/browse/HARMONY-68?page=comments#action_12366784
]
Tim Ellison commented on HARMONY-68:
------------------------------------
The test looks invalid to me. You shoud only expect an
java.nio.charset.IllegalCharsetNameException if the name itself
contains disallowed characters, and both underscore and dash are
permitted.
The code Charset.isSupported("-UTF-8")
should return false, not throw an exception.
java.nio.charset.Charset.isSupported(String charsetName) does not
throw IllegalCharsetNameException for spoiled standard sharset name
-------------------------------------------------------------------------------------------------------------------------------------
Key: HARMONY-68
URL: http://issues.apache.org/jira/browse/HARMONY-68
Project: Harmony
Type: Bug
Components: Classlib
Reporter: Svetlana Samoilenko
Attachments: charset_patch.txt
According to j2se 1.4.2 specification for Charset.isSupported(String
charsetName) the method must throw IllegalCharsetNameException "if
the given charset name is illegal ". "Legal charset name must begin
with either a letter or a digit. The test listed below shows that
there is no the exception if to insert "-" or "_" symbols before
standard sharset name, for example "-UTF-8" or "_US-ASCII".
Moreover the method returns "true" in this case.
BEA also does not throw the exception but returns "false".
Code to reproduce: import java.nio.charset.*;
public class test2 { public static void main (String[] args) {
// string starts neither a letter nor a digit boolean
sup=false; try{
sup=Charset.isSupported("-UTF-8");
System.out.println("***BAD. should be exception;
sup="+sup); sup=Charset.isSupported("_US-ASCII");
System.out.println("***BAD. should be exception;
sup="+sup); } catch (IllegalCharsetNameException e) {
System.out.println("***OK. Expected
IllegalCharsetNameException " + e); } } } Steps to
Reproduce: 1. Build Harmony (check-out on 2006-01-30) j2se subset as
described in README.txt. 2. Compile test2.java using BEA 1.4 javac
javac -d . test2.java
3. Run java using compatible VM (J9)
java -showversion test2
Output: C:\tmp>C:\jrockit-j2sdk1.4.2_04\bin\java.exe -showversion
test2 java version "1.4.2_04" Java(TM) 2 Runtime Environment,
Standard Edition (build 1.4.2_04-b05) BEA WebLogic JRockit(TM)
1.4.2_04 JVM (build ari-31788-20040616-1132-win-ia32, Native Threads,
GC strategy: parallel) ***BAD. should be exception; sup=false
***BAD. should be exception; sup=false
C:\tmp>C:\harmony\trunk\deploy\jre\bin\java -showversion test2 (c)
Copyright 1991, 2005 The Apache Software Foundation or its licensors,
as applicable. ***BAD. should be exception; sup=true
***BAD. should be exception; sup=true
Suggested junit test case:
------------------------ CharserTest.java
------------------------------------------------- import
java.nio.charset.*; import junit.framework.*; public class
CharsetTest extends TestCase { public static void main(String[]
args) { junit.textui.TestRunner.run(CharsetTest.class); }
public void test_isSupported() { boolean sup=false;
// string starts neither a letter nor a digit try{
sup=Charset.isSupported("-UTF-8");
fail("***BAD. should be exception
IllegalCharsetNameException"); } catch
(IllegalCharsetNameException e) { //expected
}
// string starts neither a letter nor a digit try{
sup=Charset.isSupported("_US-ASCII");
fail("***BAD. should be exception
IllegalCharsetNameException"); } catch
(IllegalCharsetNameException e) { //expected
}
} }