You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2013/06/04 23:27:03 UTC
svn commit: r1489621 -
/commons/proper/net/trunk/src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java
Author: sebb
Date: Tue Jun 4 21:26:31 2013
New Revision: 1489621
URL: http://svn.apache.org/r1489621
Log:
FindBugs: reliance on default charset
Modified:
commons/proper/net/trunk/src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java
Modified: commons/proper/net/trunk/src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java
URL: http://svn.apache.org/viewvc/commons/proper/net/trunk/src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java?rev=1489621&r1=1489620&r2=1489621&view=diff
==============================================================================
--- commons/proper/net/trunk/src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java (original)
+++ commons/proper/net/trunk/src/main/java/org/apache/commons/net/io/FromNetASCIIInputStream.java Tue Jun 4 21:26:31 2013
@@ -20,6 +20,9 @@ package org.apache.commons.net.io;
import java.io.IOException;
import java.io.InputStream;
import java.io.PushbackInputStream;
+import java.io.UnsupportedEncodingException;
+
+import javax.management.RuntimeErrorException;
/***
* This class wraps an input stream, replacing all occurrences
@@ -40,7 +43,11 @@ public final class FromNetASCIIInputStre
static {
_lineSeparator = System.getProperty("line.separator");
_noConversionRequired = _lineSeparator.equals("\r\n");
- _lineSeparatorBytes = _lineSeparator.getBytes();
+ try {
+ _lineSeparatorBytes = _lineSeparator.getBytes("US-ASCII");
+ } catch (UnsupportedEncodingException e) {
+ throw new RuntimeException("Broken JVM - cannot find US-ASCII charset!",e);
+ }
}
private int __length = 0;