You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by bo...@apache.org on 2009/04/15 11:32:03 UTC
svn commit: r765090 -
/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/FallbackZipEncoding.java
Author: bodewig
Date: Wed Apr 15 09:32:00 2009
New Revision: 765090
URL: http://svn.apache.org/viewvc?rev=765090&view=rev
Log:
Remove TODO, see constructor's Javadoc
Modified:
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/FallbackZipEncoding.java
Modified: commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/FallbackZipEncoding.java
URL: http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/FallbackZipEncoding.java?rev=765090&r1=765089&r2=765090&view=diff
==============================================================================
--- commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/FallbackZipEncoding.java (original)
+++ commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/FallbackZipEncoding.java Wed Apr 15 09:32:00 2009
@@ -87,7 +87,7 @@
*/
public String decode(byte[] data) throws IOException {
if (this.charset == null) {
- return new String(data); // TODO is it correct to use the default charset here?
+ return new String(data);
} else {
return new String(data,this.charset);
}