Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Received-SPF: pass (herse.apache.org: domain of wuyuehao@gmail.com designates
 66.249.82.231 as permitted sender)
DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws;
        s=beta; d=gmail.com;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=kGmcN0ocEJNDUATmhgVkNE/h6bThzagifuydJYrby3VcacGJU0Gp976NIn5QdsShP654PG6dnpflIqTFY1BL9rSw0L+VUIJydQcL8XcxnhbXbWpV9E77rcIoWWMrKjkCqwR1fDW9srtFp0w0yCOetbHhx2iPIQ1qSBLBE+qOVN8=
Message-ID: <211709bc0612262209p2211cb49i90b742657e387f14@mail.gmail.com>
Date: Wed, 27 Dec 2006 14:09:03 +0800
From: "Tony Wu" <wuyuehao@gmail.com>
To: dev@harmony.apache.org
Subject: Re: [jira] Created: (HARMONY-2862) [classlib][suncompat] ICU
 references sun.io classes that are absent
In-Reply-To: <3D8E84095C6A524A985B787423094E4095D49E@mssmsx411>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <3D8E84095C6A524A985B787423094E4095D49E@mssmsx411>

These 4 classes will not affect the behavior of harmony. It's high
time that we removed them? ;-)

I have sent a mail to ICU mailing list to ask for their opinions.

On 12/27/06, Zakharov, Vasily M <vasily.m.zakharov@intel.com> wrote:
>
> I wonder how complex a task could adding these classes be?
> They're not interfaces or fully-abstracts, they do some actual
> conversion, as one may guess from these links:
> http://icu.sourceforge.net/apiref/icu4jni/com/ibm/icu4jni/converters/Byt
> eToCharConverterICU.html
> http://icu.sourceforge.net/apiref/icu4jni/com/ibm/icu4jni/converters/Cha
> rToByteConverterICU.html
>
> From the other point of view, Harmony never had these classes, and was
> still able to run Eclipse and whatever else, so probably they're not so
> much needed. Can we ask ICU guys what those classes could be really
> needed for?
>
> And by the way, they're Deprecated.
>
> Probably we could add stubs or something like that to suncompat, that
> would throw new
> UnsupportedOperationException("sun.io.ByteToCharConverter is deprecated
> and not supported") on each method call? That's not much pain and if
> someone ever needs them, it's anyway better than NoClassDefFoundError,
> isn't it?
>
>  Vasily
>
>
> -----Original Message-----
> From: Paulex Yang [mailto:paulex.yang@gmail.com]
> Sent: Wednesday, December 27, 2006 4:29 AM
> To: dev@harmony.apache.org
> Subject: Re: [jira] Created: (HARMONY-2862) [classlib][suncompat] ICU
> references sun.io classes that are absent
>
> Geir Magnusson Jr. wrote:
> >
> > On Dec 26, 2006, at 8:12 PM, Paulex Yang wrote:
> >
> >> Geir Magnusson Jr. wrote:
> >>> ROTFL
> >>>
> >>> Who makes this library?  IBM?  :)
> >>>
> >>> I think we need to add these classes to our suncompat package...
> >>>
> >>> geir
> >>>
> >>>
> >>> On Dec 25, 2006, at 8:20 PM, Zakharov, Vasily M wrote:
> >>>
> >>>>
> >>>> I wonder what must we do to this issue.
> >>>> Any opinions?
> >> AFAIK, these four classes in ICU4JNI are not used by Harmony's
> >> nio_char module, whatever directly or indirectly(i.e., you can remove
>
> >> these classes from ICU4JNI without affecting Harmony functionality),
> >> and nio_char should be the only module which refers to ICU4JNI, so I
> >> suppose they are not needed to be loaded?
> >
> > What if a users wanted to use this package on Harmony?
> I've no objection to add sun.io.CharToByteConverter to suncompat,
> because I guess this is one of the most used Sun JDK implementation
> classes, but I think we may have other options to deal with this ICU4JNI
>
> issue, for example, remove these four classes from our build if legal,
> ICU4JNI is provided only in source codes, so we need to build it into a
> binary jar ourselves anyway.
> >
> > geir
> >
> >>>>
> >>>>  Vasily
> >>>>
> >>>>
> >>>> -----Original Message-----
> >>>> From: Vasily Zakharov (JIRA) [mailto:jira@apache.org]
> >>>> Sent: Friday, December 22, 2006 7:54 PM
> >>>> To: commits@harmony.apache.org
> >>>> Subject: [jira] Created: (HARMONY-2862) [classlib][suncompat] ICU
> >>>> references sun.io classes that are absent
> >>>>
> >>>> [classlib][suncompat] ICU references sun.io classes that are absent
> >>>> -------------------------------------------------------------------
> >>>>
> >>>>                  Key: HARMONY-2862
> >>>>                  URL:
> >>>> http://issues.apache.org/jira/browse/HARMONY-2862
> >>>>              Project: Harmony
> >>>>           Issue Type: Bug
> >>>>           Components: Classlib
> >>>>             Reporter: Vasily Zakharov
> >>>>             Priority: Minor
> >>>>
> >>>>
> >>>> DRLVM bootclasspath contains icu4jni-3.4.jar, that contains 4
> classes
> >>>> that can't be loaded. These classes are:
> >>>>
> >>>> com.ibm.icu4jni.converters.ByteToCharConverterICU extends
> >>>> sun.io.ByteToCharConverter
> >>>> com.ibm.icu4jni.converters.ByteToCharGB18030 extends
> >>>> ByteToCharConverterICU
> >>>> com.ibm.icu4jni.converters.CharToByteConverterICU extends
> >>>> sun.io.CharToByteConverter
> >>>> com.ibm.icu4jni.converters.CharToByteGB18030 extends
> >>>> CharToByteConverterICU
> >>>>
> >>>> These classes can't be loaded because they extend classes from
> sun.io
> >>>> package that are not available in Harmony classlib.
> >>>>
> >>>> There're two ways to resolve this issue:
> >>>>
> >>>> 1. Create stubs for sun.io.ByteToCharConverterICU and
> >>>> sun.io.CharToByteConverter in suncompat module.
> >>>>
> >>>> 2. Consider the four classes mentioned above non-needed for Harmony
> >>>> operation and close this issue as Won't Fix.
> >>>>
> >>>>
> >>>> --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
> >>>>
> >>>>
> >>>
> >>>
> >>
> >>
> >> --Paulex Yang
> >> China Software Development Lab
> >> IBM
> >>
> >>
> >
> >
>
>
> --
> Paulex Yang
> China Software Development Lab
> IBM
>


-- 
Tony Wu
China Software Development Lab, IBM