Author: niallp
Date: Tue Oct 5 22:00:58 2010
New Revision: 1004825
URL: http://svn.apache.org/viewvc?rev=1004825&view=rev
Log:
IO-162 Throw XmlStreamReaderException instead of IOException (looks like this
was missed when it was introduced in ROME)
Modified:
commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/XmlStreamReader.java
commons/proper/io/trunk/src/test/java/org/apache/commons/io/input/XmlStreamReaderUtilitiesTest.java
Modified:
commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/XmlStreamReader.java
URL:
http://svn.apache.org/viewvc/commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/XmlStreamReader.java?rev=1004825&r1=1004824&r2=1004825&view=diff
==============================================================================
---
commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/XmlStreamReader.java
(original)
+++
commons/proper/io/trunk/src/main/java/org/apache/commons/io/input/XmlStreamReader.java
Tue Oct 5 22:00:58 2010
@@ -514,7 +514,7 @@ public class XmlStreamReader extends Rea
if (bomEnc.equals(UTF_16BE) || bomEnc.equals(UTF_16LE)) {
if (xmlGuessEnc != null && !xmlGuessEnc.equals(bomEnc)) {
String msg = MessageFormat.format(RAW_EX_1, new Object[] {
bomEnc, xmlGuessEnc, xmlEnc });
- throw new IOException(msg);
+ throw new XmlStreamReaderException(msg, bomEnc, xmlGuessEnc,
xmlEnc);
}
if (xmlEnc != null && !xmlEnc.equals(UTF_16) &&
!xmlEnc.equals(bomEnc)) {
String msg = MessageFormat.format(RAW_EX_1, new Object[] {
bomEnc, xmlGuessEnc, xmlEnc });
Modified:
commons/proper/io/trunk/src/test/java/org/apache/commons/io/input/XmlStreamReaderUtilitiesTest.java
URL:
http://svn.apache.org/viewvc/commons/proper/io/trunk/src/test/java/org/apache/commons/io/input/XmlStreamReaderUtilitiesTest.java?rev=1004825&r1=1004824&r2=1004825&view=diff
==============================================================================
---
commons/proper/io/trunk/src/test/java/org/apache/commons/io/input/XmlStreamReaderUtilitiesTest.java
(original)
+++
commons/proper/io/trunk/src/test/java/org/apache/commons/io/input/XmlStreamReaderUtilitiesTest.java
Tue Oct 5 22:00:58 2010
@@ -135,24 +135,24 @@ public class XmlStreamReaderUtilitiesTes
// Expected BOM Guess XMLEnc Default
checkRawEncoding(bomEnc, bomEnc, null, null, defaultEnc);
checkRawEncoding(bomEnc, bomEnc, bomEnc, null, defaultEnc);
- //checkRawError(RAWMGS1, bomEnc, otherEnc, null,
defaultEnc); throws IOException for UTF-16BE/LE
+ checkRawError(RAWMGS1, bomEnc, otherEnc, null, defaultEnc);
checkRawEncoding(bomEnc, bomEnc, null, bomEnc, defaultEnc);
checkRawError(RAWMGS1, bomEnc, null, otherEnc, defaultEnc);
checkRawEncoding(bomEnc, bomEnc, bomEnc, bomEnc, defaultEnc);
checkRawError(RAWMGS1, bomEnc, bomEnc, otherEnc, defaultEnc);
- //checkRawError(RAWMGS1, bomEnc, otherEnc, bomEnc,
defaultEnc); throws IOException for UTF-16BE/LE
+ checkRawError(RAWMGS1, bomEnc, otherEnc, bomEnc, defaultEnc);
}
/** Additional UTF-16 calculateRawEncoding() Test */
public void testCalculateRawEncodingAdditonalkUTF16() throws IOException {
// BOM Guess XML
Default
-// checkRawError(RAWMGS1, "UTF-16BE", "UTF-16", null,
null); throws IOException
+ checkRawError(RAWMGS1, "UTF-16BE", "UTF-16", null, null);
checkRawEncoding("UTF-16BE", "UTF-16BE", null, "UTF-16", null);
checkRawEncoding("UTF-16BE", "UTF-16BE", "UTF-16BE", "UTF-16", null);
checkRawError(RAWMGS1, "UTF-16BE", null, "UTF-16LE", null);
checkRawError(RAWMGS1, "UTF-16BE", "UTF-16BE", "UTF-16LE", null);
-// checkRawError(RAWMGS1, "UTF-16LE", "UTF-16", null,
null); throws IOException
+ checkRawError(RAWMGS1, "UTF-16LE", "UTF-16", null, null);
checkRawEncoding("UTF-16LE", "UTF-16LE", null, "UTF-16", null);
checkRawEncoding("UTF-16LE", "UTF-16LE", "UTF-16LE", "UTF-16", null);
checkRawError(RAWMGS1, "UTF-16LE", null, "UTF-16BE", null);