commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jörg Schaible <Joerg.Schai...@scalaris.com>
Subject Re: svn commit: r1326609 - in /commons/proper/io/trunk/src: changes/changes.xml main/java/org/apache/commons/io/FileUtils.java
Date Wed, 18 Apr 2012 13:34:13 GMT
Gary Gregory wrote:

> On Wed, Apr 18, 2012 at 12:38 AM, Stefan Bodewig <bodewig@apache.org>
> wrote:
> 
>> On 2012-04-16, <ggregory@apache.org> wrote:
>>
>> > [IO-324] Add Charset sister APIs to method that take a String charset
>> name.
>>
>> The new methods cause problems for people who pass in null for the
>> charset as they want the platform's system default.  The compiler
>> doesn't know which of the writeStringToFile methods to pick if the third
>> param is null.
>>
> 
> You are correct. See the release notes:
> 
> Compatibility with 2.2 and 1.4:
> Binary compatible: Yes.
> Source compatible: No, see the rare case in
> https://issues.apache.org/jira/browse/IO-318.
> Semantic compatible: No, see the rare case in
> https://issues.apache.org/jira/browse/IO-318.
> 
> There are two solutions if you want to use the latest [io], as noted in
> the Jira:
> 
> - type-cast the charset to String to Charset, or
> - use the API that does not have a charset.

We could add a writeStringToFile() method without parameter. No need for a 
null value, no need for the cast.

- Jörg


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message