Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: harmony-dev@incubator.apache.org
Received-SPF: neutral (asf.osuosl.org: 202.81.18.152 is neither permitted nor
 denied by domain of paulex.yang@gmail.com)
Message-ID: <4411474A.9070805@gmail.com>
Date: Fri, 10 Mar 2006 17:30:50 +0800
From: Paulex Yang <paulex.yang@gmail.com>
User-Agent: Thunderbird 1.5 (Windows/20051201)
MIME-Version: 1.0
To: harmony-dev@incubator.apache.org
Subject: Re: [jira] Updated: (HARMONY-156) InputStreamReader.getEncoding()
 and OutputStreamWriter.getEncoding() should return a historical charset name.
References: <839564812.1141787621012.JavaMail.jira@ajax>
 <98e7db8d0603090540k219c68abn48b482ea4ac4415f@mail.gmail.com>
In-Reply-To: <98e7db8d0603090540k219c68abn48b482ea4ac4415f@mail.gmail.com>
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset=ISO-8859-1; format=flowed

Kononov,

Thank you to point out these. My comments below.

Dmitry M. Kononov wrote:
> Paulex,
>
> I have some thoughts about the patch. It looks good but the following two
> things:
>
> 1) java/io/InputStreamReader.
> getHistoricalName(String name) should not be public I think.
>   
the getHistoricalName(String) is not method of InputStreamReader, but a 
method of package private internal class 
InputStreamReader.HistoricalNamesUtil, so it should be OK to be public. 
And of course, it is more precise to make it package private.
> 2) java/io/OutputStreamWriter.getEncoding().
> You probably meant
>
> if (!isOpen()) {
>     return null;
> }
>
> instead of
>
> if (encoder == null) {
>     return null;
> }
>   
There is no corresponding isOpen() method in OutputStreamWriter, like 
InputStreamReader, it's weird but true:), and because the encoder is set 
to null at the close() method, so that the encoder==null is fine. And of 
course again, isOpen() is more self-descriptive than encoder==null, so 
this method should be added into OutputStreamWriter.
> since this method returns null if the stream has been closed.
>
> Thanks.
>
> On 3/8/06, Paulex Yang (JIRA) <jira@apache.org> wrote:
>   
>>     [ http://issues.apache.org/jira/browse/HARMONY-156?page=all ]
>>
>> Paulex Yang updated HARMONY-156:
>> --------------------------------
>>
>>    Attachment: patch_156.txt
>>                patch_156_test.txt
>>
>> As no one has better idea than my prior proposal, i.e., save the mapping
>> between historical name and canonical name, I created the patch for this
>> solution. The affected package is luni/java.io.
>>
>> --
>> Dmitry M. Kononov
>> Intel Managed Runtime Division
>>     
>
>   


-- 
Paulex Yang
China Software Development Lab
IBM